nextcloud / officeonline

officeonline
47 stars 14 forks source link

fix: Avoid unlocking too early and fix collaboration in Word #423

Closed juliusknorr closed 1 year ago

juliusknorr commented 1 year ago

Fix #419

Summary

It seems that OOS sends UNLOCK requests instead of GET_LOCK with GetCurrentLock-00000000-0000-0000-0000-000000000000 as a lock token just to check the current lock. Also we still need the officeonline table for storing and comparing the lock tokens, as the server locking API currently doesn't expose methods to store a custom token.

Known issues to be documented in the README:

luka-nextcloud commented 1 year ago

@juliushaertl Seems like the switch-case statements are missing default case handlers.

juliusknorr commented 1 year ago

@luka-nextcloud Expected in this case as we don't have a default path and just continue with the execution path.