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.

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

fix(carddav): Handle race for SAB creation better #42052

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix(carddav): Handle race for SAB creation better #42052

wants to merge 1 commit into from

Conversation

ChristophWurst
Copy link
Member

@ChristophWurst ChristophWurst commented Dec 6, 2023 •
edited
Loading

Summary

  • Accept non repeatable read and INSERT conflict by another read
  • Handle replication edge case

This means the code is able to handle concurrency when writing, but is (still/again) vulnerable to read/write split timing problems.

Checklist

@ChristophWurst ChristophWurst added the 2. developing Work in progress label Dec 6, 2023
@ChristophWurst ChristophWurst self-assigned this Dec 6, 2023
@ChristophWurst ChristophWurst mentioned this pull request Dec 6, 2023
Open
@susnux susnux added this to the Nextcloud 30 milestone Apr 18, 2024
@skjnldsv skjnldsv removed this from the Nextcloud 30 milestone Aug 14, 2024
@ChristophWurst ChristophWurst marked this pull request as ready for review September 17, 2024 21:58
@ChristophWurst ChristophWurst added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Sep 17, 2024
@tcitworld
Copy link
Member

Needs a rebase

@ChristophWurst ChristophWurst force-pushed the fix/carddav/create-sab-concurrently branch from a076356 to 9ba2828 Compare September 18, 2024 07:57
@ChristophWurst
fix(carddav): Handle race for SAB creation better
0e1d12b 
* Accept non repeatable read and INSERT conflict by another read
* Handle replication edge case

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst ChristophWurst force-pushed the fix/carddav/create-sab-concurrently branch from 9ba2828 to 0e1d12b Compare September 18, 2024 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@miaulalala miaulalala miaulalala approved these changes

@tcitworld tcitworld Awaiting requested review from tcitworld tcitworld is a code owner

@juliushaertl juliushaertl Awaiting requested review from juliushaertl

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

Assignees

@ChristophWurst ChristophWurst

Labels
3. to review Waiting for reviews
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

System address book creation errors on concurrent user creation
5 participants
@ChristophWurst @tcitworld @miaulalala @susnux @skjnldsv