Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quality checks #30

Merged
merged 2 commits into from
Nov 14, 2023
Merged

Quality checks #30

merged 2 commits into from
Nov 14, 2023

Conversation

eccles
Copy link
Member

@eccles eccles commented Nov 3, 2023

Check listener for nil and nil value and return error.
Check ChainHandler for nil and nil value and return error.
Check ChainHandler for returned nil and return error.
Some extra logging

AB#8685

httpserver/httpserver.go Outdated Show resolved Hide resolved
httpserver/httpserver.go Outdated Show resolved Hide resolved
startup/listener.go Outdated Show resolved Hide resolved
@eccles eccles force-pushed the dev/paul/8685-panic branch from 90a6e4a to ecacf12 Compare November 6, 2023 16:22
@eccles eccles requested a review from honourfish November 6, 2023 17:03
@eccles eccles force-pushed the dev/paul/8685-panic branch 4 times, most recently from 370e5e5 to e59759c Compare November 13, 2023 08:35
Check listener for nil and nil value and return error.
Check ChainHandler for nil and nil value and return error.
Check ChainHandler for returned nil and return error.
Some extra logging

AB#8685
@eccles eccles force-pushed the dev/paul/8685-panic branch from e59759c to e7e8fe2 Compare November 13, 2023 16:43
@eccles eccles merged commit 8b25165 into main Nov 14, 2023
1 check passed
@eccles eccles deleted the dev/paul/8685-panic branch November 14, 2023 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants