-
Notifications
You must be signed in to change notification settings - Fork 410
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
Logout with date query params #1974
Comments
We too are running into this, wondering if there is a better solution since you can't wildcard query params for logout urls |
Perhaps a config option to control whether the logout route uses the OIDC or Auth0 ( |
@seanparmelee That would work, but I was curious if that's already controlled by the logic in lines 373->390 here? I'm unsure if there's anywhere we can print the This issue started ~2 days ago for us without any code change ("@auth0/nextjs-auth0": "^4.0.2") so we weren't sure if logic under the hood has changed. ![]() |
Same exact thing happened to us! I had used the management API to double check our tenant settings (https://auth0.com/docs/api/management/v2/tenants/tenant-settings-route) and didn't see an
Then when I re-queried the tenant settings, the response included "oidc_logout": {
"rp_logout_end_session_endpoint_discovery": false
} and The "The Auth0 client does not have RP-initiated logout enabled" warning is a bit annoying though because we purposely want to use the |
That was extremely helpful, thank you so much! This worked on our staging tenant but isn't fully doing the trick on production, so there must be something else too. We're at a point now where querying both gets us "oidc_logout": { In production for some reason, it still ends up hitting |
@fezheng397 good to know; I'm curious to hear what you find out as we're not quite in prod yet with v4. |
@seanparmelee Ended up being some sort of caching issue as after a fresh deploy, it's now working and hitting |
Checklist
Describe the problem you'd like to have solved
This is particularly for NextJS SDK. Other SDK may also have this but cannot confirm.
It’s frustrating when we need to add exact pathname and query params when logging out of any application specially web application. We have a booking commerce website where the user can login after they do search. On searching, query params like start_date, end_date is added to the query params as we are maintaining the states in the URL itself. But Auth0 doesn’t allow dynamic pathname, nor dynamic params.
We also have more than 500 product page considering the locales as well and putting every url to the allowed logout urls is a mess. Also it’s a mess when we consider that we cannot pre determine the query params the users can have.
Describe the ideal solution
It would be really good if we don’t have any restriction on the redirect url after logging out so that the sdk will redirect from the page the user applied logout from.
Even if that’s not possible, atleast not restrict them on query params because it’s quite tedious and makes no sense validating the state params to auth0
Alternatives and current workarounds
As of now the current alternative is to create a page which return null but redirect the users based on the url stored in local storage which imo is a flaw in the UX and flow.
Additional context
No response
The text was updated successfully, but these errors were encountered: