1771 - Set id_token_hint when logging out

[simon-debruijn]

• All new/changed/fixed functionality is covered by tests (or N/A)
• I have added documentation for all new/changed functionality (or N/A)

📋 Changes

• Include id_token_hint in searchParams for endSessionUrl
• Ran npm run pretty
• This is a fix in behaviour and doesn't change any public or private api

📎 References

• Issue: Send id_token_hint when logging out #1771
• Duplicate PR without tests: set id_token_hint in search params when logging out if available #1773

🎯 Testing

When triggering the logout handler you can verify if the id_token_hint search param is set during the redirect.
Before this change the id_token_hint was missing. it should be present now.

[simon-debruijn]

Can someone review this please?

[frederikprijck]

Hey, apologies for not getting back to you here, unsure why this was not replied to.

Few months have passed, forgive me for asking, is this still something you need in v3, knowing we have added support for this in v4?

If yes, I can help getting this PR in.

[simon-debruijn]

Hey, apologies for not getting back to you here, unsure why this was not replied to.

Few months have passed, forgive me for asking, is this still something you need in v3, knowing we have added support for this in v4?

If yes, I can help getting this PR in.

Unfortunately the project where we need it still uses version 2.4. We're in the process of also upgrading react and nextjs versions. So at that point we can probably upgrade to v4 of nextjs-auth0.

cc @brampauwelyn What do you think?

The base branch was changed.

[tusharpandey13]

We need commits to be signed in order to merge contributor PRs. I've created a duplicate PR #2151 that contains all the commits from this PR, co-authored by me so that this change can be merged into v3 and a release be made.
Apologies for the late merge, we will merge these changes and make a v3 release soon.