Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

tusharpandey13
Copy link
Contributor

@tusharpandey13 tusharpandey13 commented Jun 9, 2025

This is a duplicate of #1780, thanks @simon-debruijn. This PR signs the original contributor's commits while preserving their contribution history to nextjs-auth0.

  • 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

🎯 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.

@tusharpandey13 tusharpandey13 requested a review from a team as a code owner June 9, 2025 04:00
@tusharpandey13 tusharpandey13 changed the base branch from main to v3 June 9, 2025 04:01
@tusharpandey13 tusharpandey13 changed the title pr 1780 1771 - Set id_token_hint when logging out (Signed) Jun 9, 2025
@tanya732
Copy link

tanya732 commented Jun 9, 2025

Browserstack test is failing

@tusharpandey13 tusharpandey13 merged commit 61c0c2c into v3 Jun 10, 2025
7 of 8 checks passed
@tusharpandey13 tusharpandey13 deleted the pr-1780 branch June 10, 2025 10:11
tusharpandey13 added a commit that referenced this pull request Jun 10, 2025
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

Successfully merging this pull request may close these issues.

3 participants