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

Add tenant OIDC logout configuration #517

Merged
merged 2 commits into from
Mar 10, 2025

Conversation

ErwinSteffens
Copy link
Contributor

@ErwinSteffens ErwinSteffens commented Feb 27, 2025

🔧 Changes

Add OIDC logout tenant settings.

📚 References

🔬 Testing

I think it is pretty trivial change. You can test setting and reading the property and see if it reflects in the UI.

I had some issues with running E2E tests locally and updating the recordings.

📝 Checklist

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

@ErwinSteffens ErwinSteffens requested a review from a team as a code owner February 27, 2025 07:52
@codecov-commenter
Copy link

codecov-commenter commented Mar 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.62%. Comparing base (5d08847) to head (aba6c69).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #517   +/-   ##
=======================================
  Coverage   95.61%   95.62%           
=======================================
  Files          57       57           
  Lines       11269    11286   +17     
=======================================
+ Hits        10775    10792   +17     
  Misses        373      373           
  Partials      121      121           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@developerkunal
Copy link
Contributor

Hi @ErwinSteffens,

I hope you're doing well!

I've updated the test records. Could you please sign your commits so we can proceed with merging the PR?

Thanks for your contribution!

@ErwinSteffens
Copy link
Contributor Author

Could you please sign your commits so we can proceed with merging the PR?

Done!

@developerkunal
Copy link
Contributor

Hi @ErwinSteffens,

I hope you're doing well!

I noticed that the commits are not signed. Could you double-check on your end?

@ErwinSteffens
Copy link
Contributor Author

I noticed that the commits are not signed. Could you double-check on your end?

Ah, just signed my commit. Did it again now.

@developerkunal developerkunal enabled auto-merge (squash) March 9, 2025 14:55
Copy link
Contributor

@developerkunal developerkunal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@developerkunal developerkunal merged commit 946df40 into auth0:main Mar 10, 2025
6 checks passed
@ErwinSteffens
Copy link
Contributor Author

@developerkunal You know when next release will be done?

@developerkunal developerkunal mentioned this pull request Mar 11, 2025
@developerkunal
Copy link
Contributor

Hi @ErwinSteffens,

Apologies for the late response, and thank you for your contribution! We released the latest version yesterday, which includes this feature. We look forward to more contributions from you in the future!

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.

4 participants