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

Backport 6.5/task 67975 97974 #220

Merged
merged 2 commits into from
Dec 6, 2023
Merged

Conversation

rdenarie
Copy link
Member

@rdenarie rdenarie commented Dec 6, 2023

No description provided.

Before this fix, sometimes, an entry in OO_EDITOR_CONFIG table is duplicated which lead to a problem when opening a document
This fix will ignore the additional entries, and keep only the first one
…h multiple users - EXO-67975

Before this fix, a race condition is possible when a lot of user edit the same document : even if a lock is set to the edited node when creating a new version, the lock allows the current lock owner to pass and then 2 thread (with the same user) can version the current node.
This lead to corrupt the node version history and lead to data loss

This commit enforce the lock so that it does not let the current lock owner to pass.
@rdenarie rdenarie requested a review from ahamdi December 6, 2023 16:18
@rdenarie rdenarie enabled auto-merge (rebase) December 6, 2023 16:18
Copy link

sonarqubecloud bot commented Dec 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

14.3% 14.3% Coverage
0.0% 0.0% Duplication

@rdenarie rdenarie merged commit 10493b8 into stable/2.5.x Dec 6, 2023
7 checks passed
@rdenarie rdenarie deleted the backport-6.5/task-67975-97974 branch December 6, 2023 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants