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

Document and conform log levels to assist customers in understanding the severity of different error codes #40

Open
markcoxsmith opened this issue Oct 9, 2020 · 5 comments

Comments

@markcoxsmith
Copy link

markcoxsmith commented Oct 9, 2020

The following customer was confused about how they find out fatal errors with connections and channels:
https://app.intercom.com/a/apps/ua39m1ld/inbox/inbox/all/conversations/38483300001514

The following customer was confused by a non-issue 80008 indicating channel disconnection. It would be helpful to outline that this is not something to be unduly worried about: https://app.intercom.com/a/apps/ua39m1ld/inbox/inbox/all/conversations/38483300001724

┆Issue is synchronized with this Jira Task by Unito

@markcoxsmith
Copy link
Author

As a result, it may be helpful for customers to have an idea of the severity of their issues so that they can easily diagnose how problematic their issue is.

@ably-sync-bot
Copy link

➤ Ivan Kavalerov commented:

This is an interesting one. When we get to this ticket I want to take a look at the ways we emit errors in different libraries - maybe there is a way to emit them in a more obvious way as well?

@mattheworiordan
Copy link
Member

The loggers, when emitting errors, always include a log level, such as warn or error. I am not sure it's practical to couple an error code to the severity as it may be different depending on the context i.e. the error code maps to a particular type of problem, but that is not necessarily an indication of the severity. A good example is a timeout, a timeout may cause a message publish to fail, or trigger a retry. With the retry, we may want to show info or major log level, especially if it subsequently succeeds, whereas an error would be emitted if it resulted in failure to publish the message.

I did see a discussion recently (I think with @QuintinWillison) on the fact that our spec does not define log levels, and instead we leave that to the developer decide and to rely on the standard log levels available in that language. This is perhaps an area we want to tighten up in the spec too.

@ably-sync-bot
Copy link

➤ Ivan Kavalerov commented:

Jamie Newcomb do you know why this stopped syncing? Looks like it may because of some tags Matt has added on the original GH issue?

@QuintinWillison QuintinWillison changed the title Potential Inclusion of Severity of Different Error Codes Document and conform log levels to assist customers in understanding the severity of different error codes Oct 12, 2020
@QuintinWillison QuintinWillison transferred this issue from ably/docs Oct 3, 2022
@sync-by-unito
Copy link

sync-by-unito bot commented Oct 17, 2022

➤ Automation for Jira commented:

The link to the corresponding Jira issue is https://ably.atlassian.net/browse/SDK-2781

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants