Skip to content

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

Merged
AndyScherzinger merged 3 commits into
stable29from
backport/51051/stable29
Mar 3, 2025
Merged

AndyScherzinger merged 3 commits into
stable29from
backport/51051/stable29

Conversation

@backportbot

@backportbot backportbot Bot commented Feb 27, 2025 •

Copy link
Copy Markdown

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
@AndyScherzinger
AndyScherzinger deleted the backport/51051/stable29 branch March 3, 2025 14:26
@blizzz blizzz mentioned this pull request Mar 4, 2025
2 of 13 tasks
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