Skip to content

fix: log memory usage for requests based on configured memory limit #57590

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.

Sign up for GitHub

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

Jump to bottom
Merged
szaimen merged 2 commits into from
Jan 29, 2026
Merged

fix: log memory usage for requests based on configured memory limit #57590

szaimen merged 2 commits into from
Jan 29, 2026
+38 −8

Conversation

@szaimen
Copy link
Contributor

@szaimen szaimen commented Jan 16, 2026 •
edited
Loading

@szaimen szaimen added this to the Nextcloud 33 milestone Jan 16, 2026
@szaimen szaimen force-pushed the enh/52997/refactor-logging branch from e00ae8f to 287fe12 Compare January 16, 2026 10:15
@szaimen szaimen added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Jan 16, 2026
@szaimen szaimen marked this pull request as ready for review January 16, 2026 10:38
@szaimen szaimen requested a review from a team as a code owner January 16, 2026 10:38
@szaimen szaimen requested review from ArtificialOwl, CarlSchwan, ChristophWurst, icewind1991, juliusknorr, susnux and yemkareems and removed request for a team January 16, 2026 10:38
@susnux susnux requested review from come-nc and removed request for yemkareems January 16, 2026 11:25
ChristophWurst
ChristophWurst approved these changes Jan 16, 2026
View reviewed changes
Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

@szaimen
Copy link
Contributor Author

szaimen commented Jan 16, 2026

CI failure is unrelated afaics

@Altahrim Altahrim added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 16, 2026
nickvergessen
nickvergessen reviewed Jan 16, 2026
View reviewed changes
@szaimen szaimen force-pushed the enh/52997/refactor-logging branch from 9d61e68 to 45d3c64 Compare January 16, 2026 13:58
@szaimen szaimen added 3. to review Waiting for reviews and removed 4. to release Ready to be released and/or waiting for tests to finish labels Jan 16, 2026
@szaimen szaimen requested a review from nickvergessen January 16, 2026 14:41
@nextcloud-bot nextcloud-bot mentioned this pull request Jan 20, 2026
Merged
@nextcloud-bot nextcloud-bot mentioned this pull request Jan 22, 2026
Merged
@szaimen
Copy link
Contributor Author

szaimen commented Jan 28, 2026 •
edited
Loading

@nickvergessen does this PR look good to you now? :)

@szaimen szaimen force-pushed the enh/52997/refactor-logging branch from 45d3c64 to b08fca0 Compare January 29, 2026 07:52
@szaimen szaimen added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 29, 2026
@szaimen szaimen enabled auto-merge January 29, 2026 07:55
@szaimen
Copy link
Contributor Author

szaimen commented Jan 29, 2026

/backport to stable33

@szaimen
Copy link
Contributor Author

szaimen commented Jan 29, 2026

/backport to stable32

@szaimen szaimen merged commit e140081 into master Jan 29, 2026
179 of 181 checks passed
@szaimen szaimen deleted the enh/52997/refactor-logging branch January 29, 2026 08:15
This was referenced Jan 29, 2026
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nickvergessen nickvergessen nickvergessen approved these changes

@ChristophWurst ChristophWurst ChristophWurst approved these changes

@Altahrim Altahrim Altahrim approved these changes

@ArtificialOwl ArtificialOwl Awaiting requested review from ArtificialOwl ArtificialOwl is a code owner automatically assigned from nextcloud/server-backend

@icewind1991 icewind1991 Awaiting requested review from icewind1991 icewind1991 is a code owner automatically assigned from nextcloud/server-backend

@CarlSchwan CarlSchwan Awaiting requested review from CarlSchwan CarlSchwan is a code owner automatically assigned from nextcloud/server-backend

@susnux susnux Awaiting requested review from susnux

@juliusknorr juliusknorr Awaiting requested review from juliusknorr

@come-nc come-nc Awaiting requested review from come-nc

+1 more reviewer

@bmarwell bmarwell bmarwell approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

4. to release Ready to be released and/or waiting for tests to finish bug enhancement

Projects

None yet

Milestone

Nextcloud 33

Development

Successfully merging this pull request may close these issues.

[Bug]: Excessive memory consumption for webdav: Request used more than 300 MB of RAM: 304.1 MB

6 participants

@szaimen @nickvergessen @ChristophWurst @bmarwell @Altahrim