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

[stable27] feat: Close sessions created for login flow v2 #51086

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
@artonge artonge force-pushed the backport/51051/stable27 branch from 57bff40 to 7942348 Compare March 3, 2025 10:39
@artonge artonge marked this pull request as ready for review March 3, 2025 10:54
artonge added 2 commits March 3, 2025 12:30
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>
Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge artonge force-pushed the backport/51051/stable27 branch from f7d0aa6 to 38f6d9a Compare March 3, 2025 11:30
Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge artonge force-pushed the backport/51051/stable27 branch from 38f6d9a to 96f0762 Compare March 3, 2025 13:44
@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 e960c0d into stable27 Mar 3, 2025
39 of 40 checks passed
@AndyScherzinger AndyScherzinger deleted the backport/51051/stable27 branch March 3, 2025 15:49
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