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 locking currently only 30min #49

Closed
tobiasKaminsky opened this issue Jan 24, 2022 · 2 comments
Closed

File locking currently only 30min #49

tobiasKaminsky opened this issue Jan 24, 2022 · 2 comments
Labels
question Further information is requested
Milestone

Comments

@tobiasKaminsky
Copy link
Member

As dicussced in nextcloud/server#21914 I assumed that file locking will be "forever", but Maxence said that current state locks file only for 30min, which could be changed to a config value.
This means that longer editing of a file is not possible.
On the other hand it means that we do not need to have any unlock mechanism via admin.

@AndyScherzinger @schiessle what is your opinion here?

@tobiasKaminsky tobiasKaminsky added the question Further information is requested label Jan 24, 2022
@AndyScherzinger
Copy link
Member

AndyScherzinger commented Jan 24, 2022

Well I'd be fine either way.

  • lock forever but rather hard-coded or the config value would have have a quite high lower-boundry because, see option 2
  • lock-time via config: clients now not just need to handle lock/unlock but also deal with a "lock-steal" situation, where lock time is e.g. 4h and a user works on the doc for 6h, tries to save, but no another used grabbed the lock after the file auto-unlocked. Also then the discussion will start that the client should "update the lock-timeframe" whenever a save is triggers or other ideas.

So my take is

  • change it to "forever" as the default
  • state in the config that other values are possible but handling isn't supported by clients
  • implement admin unlock since you face the same situation with a lock set to 3+ weeks for example: Admin needs to unlock files #52

@juliusknorr
Copy link
Member

lock-time via config: clients now not just need to handle lock/unlock but also deal with a "lock-steal" situation, where lock time is e.g. 4h and a user works on the doc for 6h, tries to save, but no another used grabbed the lock after the file auto-unlocked. Also then the discussion will start that the client should "update the lock-timeframe" whenever a save is triggers or other ideas.

I'm wondering if we should not still consider such scenarios, as the lock might also be removed by an admin and then clients should be able to handle such lock-steal scenarios anyways in some way. I could imagine that they could implement similar behaviour as a conflicting file would have on the desktop client, where a copy is preserved for manual resolution.

Nevertheless, support for infinite lock timeout is started in #55 just the decision on the default I would make dependent on above considerations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants