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

Uniform log messages #855

Merged
merged 11 commits into from
Aug 17, 2022
Merged

Uniform log messages #855

merged 11 commits into from
Aug 17, 2022

Conversation

lexnv
Copy link
Contributor

@lexnv lexnv commented Aug 16, 2022

This PR normalizes the published log messages.

Errors are logged for any unrecoverable issues (ie, terminating a client/server
connection caused via error exit paths).

While at it, clean up some comments and documentation.

Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
@lexnv lexnv requested a review from a team as a code owner August 16, 2022 18:30
@lexnv lexnv self-assigned this Aug 16, 2022
lexnv and others added 2 commits August 17, 2022 12:59
Co-authored-by: Niklas Adolfsson <niklasadolfsson1@gmail.com>
Co-authored-by: Niklas Adolfsson <niklasadolfsson1@gmail.com>
@lexnv lexnv merged commit bd31557 into master Aug 17, 2022
@lexnv lexnv deleted the clean_logs branch August 17, 2022 10:21
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.

None yet

4 participants