Skip to content

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

Merged
AndyScherzinger merged 3 commits into
stable27from
backport/51051/stable27
Mar 3, 2025
Merged

[stable27] feat: Close sessions created for login flow v2#51086
AndyScherzinger merged 3 commits into
stable27from
backport/51051/stable27

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
@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
Comment thread lib/private/Authentication/Login/Chain.php Fixed
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
@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.

3 participants