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

[stable29] feat: Close sessions created for login flow v2 #51088

Merged
merged 3 commits into from
Mar 3, 2025

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Feb 27, 2025

Backport of #51051

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot added bug 3. to review Waiting for reviews php Pull requests that update Php code labels Feb 27, 2025
@backportbot backportbot bot added this to the Nextcloud 29.0.13 milestone Feb 27, 2025
@artonge artonge force-pushed the backport/51051/stable29 branch from aab6e95 to c68656a Compare February 27, 2025 11:54
Sessions created during the login flow v2 should be short lived to not leave an unexpected opened session in the browser.

This commit add a property to the session object to track its origin, and will close it as soon as possible, i.e., on the first non public page request.

Signed-off-by: Louis Chemineau <louis@chmn.me>

[skip ci]

Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge artonge force-pushed the backport/51051/stable29 branch from c68656a to ff5a03e Compare March 3, 2025 10:39
artonge added 2 commits March 3, 2025 11:42
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge artonge marked this pull request as ready for review March 3, 2025 10:54
@artonge artonge added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 3, 2025
@AndyScherzinger AndyScherzinger merged commit dfc5dee into stable29 Mar 3, 2025
186 checks passed
@AndyScherzinger AndyScherzinger deleted the backport/51051/stable29 branch March 3, 2025 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug php Pull requests that update Php code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants