-
Notifications
You must be signed in to change notification settings - Fork 25
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
[#172038263] Add Express Error Middleware, catch passport exceptions #632
Conversation
Affected stories
Generated by 🚫 dangerJS |
@@ -29,10 +29,20 @@ const bearerSessionTokenStrategy = ( | |||
) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what about bearerWalletTokenStrategy ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in 67317c1
Co-Authored-By: Danilo Spinelli <gunzip@users.noreply.github.com>
Co-Authored-By: Danilo Spinelli <gunzip@users.noreply.github.com>
Co-Authored-By: Danilo Spinelli <gunzip@users.noreply.github.com>
Co-Authored-By: Danilo Spinelli <gunzip@users.noreply.github.com>
Codecov Report
@@ Coverage Diff @@
## master #632 +/- ##
==========================================
- Coverage 82.42% 82.37% -0.05%
==========================================
Files 51 53 +2
Lines 1485 1515 +30
Branches 247 254 +7
==========================================
+ Hits 1224 1248 +24
- Misses 250 256 +6
Partials 11 11
Continue to review full report at Codecov.
|
No description provided.