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

Handle error=unauthorized & error_description=Access denied response #54

Open
dniel opened this issue Mar 8, 2019 · 0 comments
Open
Assignees
Labels
bug Something isn't working

Comments

@dniel
Copy link
Owner

dniel commented Mar 8, 2019

Implement error handling so that unauthorized responses is handled correctly by ForwardAuth.
Now it just print the error message with the generic error json printer.
It should show an better unauthorized error page, maybe just report 401 and let Traefik route it to correct webapp to handle 401 status messages.

https://redacted/oauth2/signin?error=unauthorized&error_description=Access%20denied.&state=eyJvcmlnaW5VcmwiOnsicHJvdG9jb2wiOiJodHRwcyIsImhvc3QiOiJ0cmFlZmlrLmRuaWVsLnNlIiwidXJpIjoiL2Rhc2hib2FyZC8ifSwibm9uY2UiOnsidmFsdWUiOiJmODE2NmNkNGFlYTc0ZTNlYjEzZGI2Y2M1Y2UyNjk0ZiJ9fQ%3D%3D

@dniel dniel added the bug Something isn't working label Mar 8, 2019
dniel added a commit that referenced this issue Mar 14, 2019
When response to the signin endpoint is error=unauthorized the endpoint
will return http status 403 to the client. This could for example be
used to use the Custom error page in Traefik to configure a backend to
use to show a custom error page to the user.
dniel added a commit that referenced this issue Mar 16, 2019
dniel added a commit that referenced this issue Mar 16, 2019
dniel added a commit that referenced this issue Mar 16, 2019
dniel added a commit that referenced this issue Mar 16, 2019
dniel added a commit that referenced this issue Mar 16, 2019
dniel added a commit that referenced this issue Mar 16, 2019
dniel added a commit that referenced this issue Mar 16, 2019
@dniel dniel self-assigned this Mar 25, 2019
dniel added a commit that referenced this issue May 23, 2019
dniel added a commit that referenced this issue May 28, 2019
use only docker provider for dynamic configuration
dniel added a commit that referenced this issue May 28, 2019
dniel added a commit that referenced this issue Jun 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant