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

Improve output of WebPush error #1463

Merged
merged 1 commit into from Aug 26, 2017
Merged

Improve output of WebPush error #1463

merged 1 commit into from Aug 26, 2017

Conversation

astorije
Copy link
Member

Without this, the error displayed contains a lot of superfluous information, repeated.

Before:

screen shot 2017-08-26 at 11 46 03

The after should just be something like:

2017-08-26 06:15:45 [ERROR] WebPush Error (getaddrinfo ENOTFOUND fcm.googleapis.com fcm.googleapis.com:443)

This is similar to that and makes for a more digestible output.

Without this, the error displayed contains a lot of superfluous information, repeated.
@astorije astorije added the Type: Feature Tickets that describe a desired feature or PRs that add them to the project. label Aug 26, 2017
@astorije astorije added this to the 2.5.0 milestone Aug 26, 2017
@astorije astorije requested a review from xPaw August 26, 2017 15:49
@astorije astorije merged commit 330bb9a into master Aug 26, 2017
@astorije astorije deleted the astorije/better-error branch August 26, 2017 16:03
matburnham pushed a commit to matburnham/lounge that referenced this pull request Sep 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature Tickets that describe a desired feature or PRs that add them to the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants