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

Error handling not working properly when upgrading lib #117

Open
4integration opened this issue Jan 30, 2025 · 0 comments
Open

Error handling not working properly when upgrading lib #117

4integration opened this issue Jan 30, 2025 · 0 comments

Comments

@4integration
Copy link

Hi,

I am testing passport-openidconnect with an example app based on this https://github.com/ForgeRock/exampleOAuth2Clients/tree/master/node-passport-openidconnect

Since it is a bit dated I have updated all libs. But when updating passport-openidconnect from 0.0.2 to 0.1.2 the error handling stops working.

I have a error case when client_id is missing in Audience and with ver 0.0.2 I get a proper error handling with error page showing
aud parameter does not include this client - is: urn:myorg:user-store:test-application| expected: my-test-client

When updating the lib to ver 0.1.2 (only that change) I get
Not Found 404, NotFoundError: Not Found i.e. cannot load the error page

Do you have any idea why it fails?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant