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

[stable28] feat: Close sessions created for login flow v2 #51087

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
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 <[email protected]>

[skip ci]

Signed-off-by: Louis Chemineau <[email protected]>
@artonge artonge force-pushed the backport/51051/stable28 branch from 112b585 to 716b081 Compare March 3, 2025 10:39
@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
@artonge artonge force-pushed the backport/51051/stable28 branch from afae0d8 to 61f3bbc Compare March 3, 2025 11:50
@artonge artonge force-pushed the backport/51051/stable28 branch from 61f3bbc to f284df0 Compare March 3, 2025 13:40
@AndyScherzinger AndyScherzinger merged commit 94c54e4 into stable28 Mar 3, 2025
55 of 56 checks passed
@AndyScherzinger AndyScherzinger deleted the backport/51051/stable28 branch March 3, 2025 15:48
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