Skip to content

perf(dav): fetch federated address book cards with addressbook-multiget - #65942

Open
SebastianKrupinski wants to merge 1 commit into
fix/federation-sync-part-1-transactionsfrom
fix/federation-sync-part-3-multiget
Open

perf(dav): fetch federated address book cards with addressbook-multiget#65942
SebastianKrupinski wants to merge 1 commit into
fix/federation-sync-part-1-transactionsfrom
fix/federation-sync-part-3-multiget

Conversation

@SebastianKrupinski

@SebastianKrupinski SebastianKrupinski commented Oct 26, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Resolves: # Client ticket
  • Federated address book sync now fetches changed cards with addressbook-multiget REPORTs in batches of 100. Before, it sent one GET per card.
  • Before 10,000 cards meant 10,000 HTTP requests, each one a full bootstrap and auth on the remote. Now it's about 100 requests.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@SebastianKrupinski SebastianKrupinski added this to the Nextcloud 36 milestone Oct 26, 2026
@SebastianKrupinski SebastianKrupinski self-assigned this Oct 26, 2026
@SebastianKrupinski SebastianKrupinski added the 3. to review Waiting for reviews label Oct 26, 2026
@SebastianKrupinski
SebastianKrupinski requested review from a team and hamza221 as code owners October 26, 2026 17:03
@SebastianKrupinski
SebastianKrupinski requested review from Altahrim, leftybournes, salmart-dev and sorbaugh and removed request for a team October 26, 2026 17:03
@github-project-automation github-project-automation Bot moved this to 🏗️ In progress in 💌 📅 👥 Groupware team Oct 26, 2026
@SebastianKrupinski
SebastianKrupinski added this pull request to stack #65943 October 26, 2026 17:04
@SebastianKrupinski

SebastianKrupinski commented Oct 26, 2026

Copy link
Copy Markdown
Contributor Author

/backport to stable35

@SebastianKrupinski

SebastianKrupinski commented Oct 26, 2026

Copy link
Copy Markdown
Contributor Author

/backport to stable34

@SebastianKrupinski

SebastianKrupinski commented Oct 26, 2026

Copy link
Copy Markdown
Contributor Author

/backport to stable33

@SebastianKrupinski
perf(dav): fetch federated address book cards with addressbook-multiget
d18ee25 
Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>
@SebastianKrupinski
SebastianKrupinski force-pushed the fix/federation-sync-part-3-multiget branch from 428d8f9 to d18ee25 Compare October 26, 2026 17:28

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hamza221 hamza221 Awaiting requested review from hamza221 hamza221 is a code owner
@Altahrim Altahrim Awaiting requested review from Altahrim Altahrim is a code owner automatically assigned from nextcloud/server-backend
@salmart-dev salmart-dev Awaiting requested review from salmart-dev salmart-dev is a code owner automatically assigned from nextcloud/server-backend
@leftybournes leftybournes Awaiting requested review from leftybournes leftybournes is a code owner automatically assigned from nextcloud/server-backend
@sorbaugh sorbaugh Awaiting requested review from sorbaugh sorbaugh is a code owner automatically assigned from nextcloud/server-backend
@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst ChristophWurst is a code owner
@tcitworld tcitworld Awaiting requested review from tcitworld tcitworld is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

@SebastianKrupinski SebastianKrupinski

Labels

3. to review Waiting for reviews backport-request

Projects

💌 📅 👥 Groupware team
Status: 🏗️ In progress

Milestone

Nextcloud 36

Development

Successfully merging this pull request may close these issues.

1 participant

@SebastianKrupinski