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

Fix 404 for Unauthorized! #3

Open
meysamhadeli opened this issue Apr 30, 2023 · 7 comments
Open

Fix 404 for Unauthorized! #3

meysamhadeli opened this issue Apr 30, 2023 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@meysamhadeli
Copy link
Owner

Debug exception handlers and identity provider.

@meysamhadeli meysamhadeli added the bug Something isn't working label Apr 30, 2023
@meysamhadeli meysamhadeli self-assigned this Apr 30, 2023
@rahiyansafz
Copy link

is it fixed yet?

@meysamhadeli
Copy link
Owner Author

is it fixed yet?

Hi, Not yet. I have plan to fix this issue, but unfortunately I didn't have time to investigate with this problem. Feel free to fix this issue if you have time :)

@Jay-Dee
Copy link

Jay-Dee commented Jul 25, 2024

Hello, this seems related to AuthenticationSchemes setup. I have tested it for a single endpoint and can see a 401 as expected

14:17:02 [DBUG] Error Url: /home/error
14:17:02 [DBUG] Error Id Parameter: errorId
14:17:02 [INFO] HTTP GET /api/v1/flight/1 responded 401 in 245.2289 ms

I can raise a PR if this sounds reasonable

@meysamhadeli
Copy link
Owner Author

Hello, this seems related to AuthenticationSchemes setup. I have tested it for a single endpoint and can see a 401 as expected

14:17:02 [DBUG] Error Url: /home/error
14:17:02 [DBUG] Error Id Parameter: errorId
14:17:02 [INFO] HTTP GET /api/v1/flight/1 responded 401 in 245.2289 ms

I can raise a PR if this sounds reasonable

Hi, Thanks for comment. Please send a PR for your solution :)

@Jay-Dee
Copy link

Jay-Dee commented Jul 30, 2024

I do not have sufficient permissions for a push

@meysamhadeli
Copy link
Owner Author

I do not have sufficient permissions for a push

You have to fork the project and make a pull request.

Jay-Dee added a commit to Jay-Dee/booking-modular-monolith that referenced this issue Sep 4, 2024
@meysamhadeli
Copy link
Owner Author

@Jay-Dee
Hi,
Could you please create a PR for fixing this issue?
The latest changes here bugfix/setup-global-authentication-scheme that you mentioned is ok.

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

3 participants