Skip to content

perf: log slow DNS operations - #55265

Merged
Altahrim merged 2 commits into
masterfrom
perf/log-slow-dns
Jan 8, 2026
Merged

perf: log slow DNS operations#55265
Altahrim merged 2 commits into
masterfrom
perf/log-slow-dns

Conversation

@ChristophWurst

@ChristophWurstChristophWurst commented Sep 23, 2025•
edited
Loading

Copy link
Copy Markdown
Member

Summary

DNS can be a bottleneck. This makes the problem visible in the log when the DNS pinning middleware is slowing things down.

TODO

  • Refactor session monitoring into generic trait
  • Wrap dns operation in middleware

Checklist

@ChristophWurstChristophWurst added this to the Nextcloud 32 milestone Sep 23, 2025
@ChristophWurstChristophWurst self-assigned this Sep 23, 2025
@ChristophWurst
ChristophWurst requested a review from a team as a code ownerSeptember 23, 2025 07:44
@ChristophWurst
ChristophWurst removed the request for review from a teamSeptember 23, 2025 07:44
@ChristophWurstChristophWurst mentioned this pull request Sep 23, 2025
Closed
@ChristophWurstChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Sep 23, 2025
@ChristophWurst

ChristophWurst commented Sep 23, 2025

Copy link
Copy Markdown
MemberAuthor

/backport to stable32

This was referenced Sep 25, 2025
Merged
Merged
@ChristophWurstChristophWurst mentioned this pull request Sep 27, 2025
Closed
@nextcloud-botnextcloud-bot mentioned this pull request Jan 7, 2026
Merged
@ChristophWurst
refactor: extract slow operation logging into trait
cc89a2a
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst
perf: log slow DNS operations
adf7ea5
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@Altahrim
Altahrim merged commit 36b0d7c into masterJan 8, 2026
226 of 246 checks passed
@Altahrim
Altahrim deleted the perf/log-slow-dns branch January 8, 2026 13:06
@backportbotbackportbotBot mentioned this pull request Jan 8, 2026
Merged
@ChristophWurstChristophWurst mentioned this pull request Jan 9, 2026
Merged
7 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AltahrimAltahrimAltahrim approved these changes
@CarlSchwanCarlSchwanCarlSchwan approved these changes
@yemkareemsyemkareemsAwaiting requested review from yemkareemsyemkareems is a code owner automatically assigned from nextcloud/server-backend
@leftybournesleftybournesAwaiting requested review from leftybournesleftybournes is a code owner automatically assigned from nextcloud/server-backend

Assignees

@ChristophWurstChristophWurst

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugperformance 🚀

Projects

None yet

Milestone

Nextcloud 33

Development

Successfully merging this pull request may close these issues.

4 participants

@ChristophWurst@Altahrim@CarlSchwan@skjnldsv