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

docs(identity): Add note regarding single sign out limitation #3500

Merged
merged 2 commits into from
Apr 1, 2024

Conversation

Ben-Sheppard
Copy link
Contributor

@Ben-Sheppard Ben-Sheppard commented Mar 21, 2024

Closes https://github.com/camunda-cloud/identity/issues/2701

Description

Through the investigation and implementation of an upcoming piece of work we discovered that there are known limitations in the OpenID spec for front channel logout due to third party cookie settings.

This is something that we should make the customers aware of. The OpenID spec lists the limitation here and an example of this in the Microsoft docs is here

When should this change go live?

  • This change is not yet live and should not be merged until {ADD_DATE} (apply hold label or convert to draft PR)?
  • There is no urgency with this change.
  • This change or page is part of a marketing blog, conference talk, or something else on a schedule.
  • This functionality is already available but undocumented.
  • This is a bug fix or security concern.

PR Checklist

  • I have added changes to the relevant /versioned_docs directory, or they are not for an already released version.
  • I have added changes to the main /docs directory (aka /next/), or they are not for future versions.
  • My changes require an Engineering review, and I've assigned an engineering manager or tech lead as a reviewer, or my changes do not require an Engineering review.
  • My changes require a technical writer review, and I've assigned @christinaausley as a reviewer, or my changes do not require a technical writer review.

@Ben-Sheppard Ben-Sheppard added the component:identity Issues related with Identity project label Mar 21, 2024
@Ben-Sheppard Ben-Sheppard self-assigned this Mar 21, 2024
@Ben-Sheppard Ben-Sheppard requested review from maryarm, christinaausley and a user March 21, 2024 11:34
Copy link
Contributor

github-actions bot commented Mar 21, 2024

👋 🤖 ✅ Looks like the changes were ported across versions, nice job! 🎉

You can read more about the versioning within our docs in our documentation guidelines.

@christinaausley christinaausley added the component:self-managed Docs and issues related to Camunda Platform 8 Self-Managed label Mar 22, 2024
@christinaausley
Copy link
Contributor

This functionality is already available but undocumented.

Should this be backported, or is this only for the next version of documentation?

ghost
ghost previously approved these changes Mar 25, 2024
@Ben-Sheppard
Copy link
Contributor Author

This functionality is already available but undocumented.

Should this be backported, or is this only for the next version of documentation?

Good question @christinaausley - as OIDC was introduced in 8.4 then I should backport these changes there, I'll handle that now

@Ben-Sheppard Ben-Sheppard requested a review from a user March 25, 2024 08:52
@Ben-Sheppard
Copy link
Contributor Author

Done @christinaausley :)

@christinaausley christinaausley merged commit 7479905 into main Apr 1, 2024
7 checks passed
@christinaausley christinaausley deleted the 2701-document-single-sign-out-limitations branch April 1, 2024 13:55
theburi pushed a commit that referenced this pull request Apr 3, 2024
* docs(identity): Add note regarding single sign out limitation

* docs(identity): backport to 8.4 docs regarding single sign out limitation
theburi pushed a commit that referenced this pull request Jun 5, 2024
* docs(identity): Add note regarding single sign out limitation

* docs(identity): backport to 8.4 docs regarding single sign out limitation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:identity Issues related with Identity project component:self-managed Docs and issues related to Camunda Platform 8 Self-Managed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants