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

[NEW] Allow configuration of SAML logout behavior #9524

Closed
wants to merge 397 commits into from

Conversation

mrsimpson
Copy link
Collaborator

@mrsimpson mrsimpson commented Jan 26, 2018

@RocketChat/core

This PR addresses a TODO in the current implementation of the SAML logout handling. It allows the admin to configure what should happen once a user logged-in via SAML logs out: whether the SAML session or only the Rocket.Chat-session be terminated.

Closes #9522

Decided to make it configurable not with a boolean value, but with a select.
This allows for more flexibility handling the logout, e. g. asking the user what he'd like to log-off from.

ruKurz and others added 30 commits June 11, 2017 12:11
# Conflicts:
#	packages/rocketchat-google-natural-language/.npm/package/npm-shrinkwrap.json
#	packages/rocketchat-ui-sidenav/client/createCombinedFlex.html
#	packages/rocketchat-ui-sidenav/client/createCombinedFlex.js
Merging current Rocket.Chat 0.55.1
- Removed unnecessary code for setting a custom tracking url
- Make the list of domains editable
- Changed comment
#15 - Add advanced configuration options for piwik analytics.
- Fixed formatting issues according to stylelint
#20 The CSS format for showing the knowledge base is broken.
An expert channel which was configured as private channel prevented startup
Making release for beta test.
…o release/0.55.1-0.2

Merging hotfix from master into release branch.
- add new labels
@CLAassistant
Copy link

CLAassistant commented Jan 27, 2018

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
4 out of 5 committers have signed the CLA.

✅ ruKurz
✅ vickyokrm
✅ paulovitin
✅ mrsimpson
❌ henrikscholl


henrikscholl seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@RocketChat RocketChat deleted a comment Jan 27, 2018
@RocketChat RocketChat deleted a comment Jan 27, 2018
@RocketChat RocketChat deleted a comment Jan 27, 2018
@RocketChat RocketChat deleted a comment Jan 27, 2018
@RocketChat RocketChat deleted a comment Jan 27, 2018
@RocketChat RocketChat deleted a comment Jan 27, 2018
@RocketChat RocketChat deleted a comment Jan 27, 2018
@RocketChat RocketChat deleted a comment Jan 27, 2018
@RocketChat RocketChat deleted a comment Jan 27, 2018
@RocketChat RocketChat deleted a comment Jan 27, 2018
@RocketChat RocketChat deleted a comment Jan 27, 2018
@mrsimpson
Copy link
Collaborator Author

damn it - pushed -f to the wrong branch - will create a new one

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.

Don't kill SAML-session on logout
8 participants