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

File in collabora opens in read-only mode for the owner #3936

Closed
ScharfViktor opened this issue Jun 8, 2022 · 5 comments
Closed

File in collabora opens in read-only mode for the owner #3936

ScharfViktor opened this issue Jun 8, 2022 · 5 comments
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug

Comments

@ScharfViktor
Copy link
Contributor

Test with wopi:

Actual result: file opens in read-only permissions mode

Screenshot 2022-06-08 at 18 01 32

ocis_wopi_collabora log

wsd-00008-00404 2022-06-08 16:05:47.519043 [ docbroker_017 ] WRN  Un-successful WOPI::Lock with status 409, failure reason: "The file is locked by Collabora" and response: {"message": "The file is locked by Collabora"}| wsd/Storage.cpp:977

wsd-00008-00404 2022-06-08 16:05:47.519195 [ docbroker_017 ] ERR  Failed to lock!| wsd/DocumentBroker.cpp:882
@ScharfViktor ScharfViktor mentioned this issue Jun 8, 2022
32 tasks
@ScharfViktor ScharfViktor changed the title File opens in read-only mode for the owner File in collabora opens in read-only mode for the owner Jun 9, 2022
@wkloucek
Copy link
Contributor

Looks like a locking race condition. The first Collabora session holds the lock and did not yet release it because it didn't yet save the file. Then the second Collabora session already tries to open the still locked file and fails.

We should at least investigate in how we can improve this.

@micbar
Copy link
Contributor

micbar commented Jul 13, 2022

Needs investigation. The unlock should happen.

@micbar micbar added the Priority:p3-medium Normal priority label Jul 13, 2022
@wkloucek
Copy link
Contributor

Needs investigation. The unlock should happen.

The unlock happens. But not as fast as the user opens again that file. And since the session is currently closing, you can't just resume the edit.

@micbar micbar added this to the 2.0.0 General Availability milestone Jul 18, 2022
@micbar micbar added GA-Blocker Priority:p2-high Escalation, on top of current planning, release blocker and removed Priority:p3-medium Normal priority labels Sep 14, 2022
@C0rby
Copy link
Contributor

C0rby commented Sep 26, 2022

I can't reproduce this with the current master.
When I try to close and open the file quickly then collabora show "cleaning up last session" before opening the file as usual.

@ScharfViktor
Copy link
Contributor Author

Couldn't reproduce it. no log in collabora

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug
Projects
Archived in project
Development

No branches or pull requests

4 participants