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] perf(session): remove useless session open #46168

Merged
merged 1 commit into from
Jun 27, 2024

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Jun 27, 2024

Backport of PR #46059

`UseSession` isn’t used in theses controllers.
Not using it should avoid a few session locks.

Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
@backportbot backportbot bot added enhancement 3. to review Waiting for reviews php Pull requests that update Php code performance 🚀 labels Jun 27, 2024
@backportbot backportbot bot added this to the Nextcloud 29.0.4 milestone Jun 27, 2024
@Altahrim Altahrim enabled auto-merge June 27, 2024 09:18
@Altahrim Altahrim merged commit 01c0e74 into stable29 Jun 27, 2024
165 of 168 checks passed
@Altahrim Altahrim deleted the backport/46059/stable29 branch June 27, 2024 10:40
@Altahrim Altahrim mentioned this pull request Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement performance 🚀 php Pull requests that update Php code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants