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

Add debug logs for OnlyOfficeEditor #216

Merged
merged 1 commit into from
Dec 1, 2023
Merged

Add debug logs for OnlyOfficeEditor #216

merged 1 commit into from
Dec 1, 2023

Conversation

rdenarie
Copy link
Member

@rdenarie rdenarie commented Dec 1, 2023

No description provided.

@rdenarie rdenarie requested review from ahamdi, mkrout and azayati December 1, 2023 16:10
@rdenarie rdenarie enabled auto-merge (squash) December 1, 2023 16:10
@rdenarie rdenarie disabled auto-merge December 1, 2023 16:56
@rdenarie rdenarie enabled auto-merge (squash) December 1, 2023 16:56
Copy link

sonarqubecloud bot commented Dec 1, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

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

25.3% 25.3% Coverage
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@rdenarie rdenarie merged commit 0e2e507 into develop Dec 1, 2023
3 of 4 checks passed
@rdenarie rdenarie deleted the add-debug-logs branch December 1, 2023 18:48
rdenarie added a commit that referenced this pull request Dec 6, 2023
bourasom pushed a commit that referenced this pull request Dec 6, 2023
Jihed525 pushed a commit that referenced this pull request Dec 6, 2023
hbenali pushed a commit that referenced this pull request Mar 4, 2024
* Add debug logs for OnlyOfficeEditor (#216)

(cherry picked from commit 8912b80)

* fix: Unable to open a document in edition - EXO-67974

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

(cherry picked from commit 745a803)

* fix: Re-enforce lock when versionning the node in case of edition with 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.

(cherry picked from commit b55d5d9)

* patch-6.4.4-1

---------

Co-authored-by: Romain Dénarié <[email protected]>
Co-authored-by: jihed525 <[email protected]>
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