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

Clarified what user data remains on logout #6572

Merged
merged 23 commits into from
Aug 28, 2023
Merged

Conversation

cwarnermm
Copy link
Member

Documentation for: #6177

@cwarnermm cwarnermm added this to the v8.1.0 milestone Aug 2, 2023
@github-actions
Copy link

github-actions bot commented Aug 2, 2023

Newest code from cwarnermm has been published to preview environment for Git SHA ea063ba

@cwarnermm cwarnermm added the 1: Dev Review Requires review by a core commiter label Aug 8, 2023
source/welcome/log-out.rst Outdated Show resolved Hide resolved
@github-actions
Copy link

Newest code from cwarnermm has been published to preview environment for Git SHA 9b2f7bd

@cwarnermm cwarnermm removed this from the v8.1.0 milestone Aug 15, 2023
@cwarnermm cwarnermm changed the base branch from v8.1-documentation to master August 15, 2023 13:16
@github-actions
Copy link

Newest code from cwarnermm has been published to preview environment for Git SHA 243a52d

--------------------------

What happens when I log out of Mattermost?
Copy link
Member

Choose a reason for hiding this comment

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

I'm pretty sure most of these only apply to mobile. I can't 100% speak for the desktop app, but most of these things don't exist in the web app or aren't stored in a way that lasts through a refresh

Copy link
Member

Choose a reason for hiding this comment

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

The Desktop App would take on the same behaviour as the webapp in this case as far as I would know


When you log out, the following additional data is also deleted:
- On MacOS, your user credentials for this server from the keychain.
Copy link
Member

Choose a reason for hiding this comment

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

In particular, I don't think we explicitly interact with the keychain even from the desktop app

Copy link
Member

Choose a reason for hiding this comment

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

This is correct, there's no keychain added for web or desktop afaik.

@github-actions
Copy link

Newest code from cwarnermm has been published to preview environment for Git SHA d4d3800

@github-actions
Copy link

Newest code from cwarnermm has been published to preview environment for Git SHA ffe06c5

@cwarnermm cwarnermm added 3: Reviews Complete All reviewers have approved the pull request and removed 1: Dev Review Requires review by a core commiter 2. SME Review labels Aug 28, 2023
@cwarnermm cwarnermm merged commit 737b235 into master Aug 28, 2023
3 checks passed
@cwarnermm cwarnermm deleted the logout-clarifications branch August 28, 2023 17:49
@github-actions
Copy link

Newest code from cwarnermm has been published to preview environment for Git SHA abe3026

@github-actions
Copy link

Newest code from cwarnermm has been published to preview environment for Git SHA 02412f4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants