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.

Sign up for GitHub

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

Jump to bottom

GatewayAuthorizationMiddleware should return proper status. #37

Closed
vojtechrojicek opened this issue Jul 11, 2019 · 0 comments · Fixed by #49
Closed

GatewayAuthorizationMiddleware should return proper status. #37

vojtechrojicek opened this issue Jul 11, 2019 · 0 comments · Fixed by #49

Comments

@vojtechrojicek
Copy link
Collaborator

vojtechrojicek commented Jul 11, 2019 •
edited by Burgyn
Loading

GatewayAuthorizationMiddleware now returns only 401 (Unauthorized) for all errors during getting JWTtoken. It would be helpfull to return also another errors (NotFound, Forbidden, ... ).
@satano satano added the area-AspNetCore Kros.AspNetCore library label Aug 15, 2019
@wosciatko wosciatko mentioned this issue Aug 28, 2019
Merged
@ghost ghost added the Pending PR label Aug 28, 2019
@Burgyn Burgyn closed this as completed in #49 Sep 2, 2019
@ghost ghost removed Pending PR area-AspNetCore Kros.AspNetCore library good first issue Good for newcomers up-for-grabs labels Sep 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added specific exceptions to GatewayAuthorizationMiddleware
3 participants
@satano @Burgyn @vojtechrojicek