Skip to content

Commit

Permalink
Merge pull request #2725 from owncloud/accept_locking_adr
Browse files Browse the repository at this point in the history
[docs-only] accept locking adr
  • Loading branch information
micbar authored Nov 4, 2021
2 parents 52e5eff + 1ac831a commit ec066b8
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions docs/ocis/adr/0013-locking.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,9 @@ geekdocEditPath: edit/master/docs/ocis/adr
geekdocFilePath: 0013-locking.md
---

- Status: proposed
- Deciders: @pmaier1, @dragotin, @butonic, @micbar, @wkloucek
- Date: 2021-10-28
- Status: accepted
- Deciders: @hodyroff, @pmaier1, @jojowein, @dragotin, @micbar, @tbsbdr, @wkloucek
- Date: 2021-11-03

## Context and Problem Statement

Expand All @@ -28,7 +28,9 @@ At the time of this writing no locking mechanisms exists in oCIS / REVA for both

## Decision Outcome

// TODO, but recommendation against file based locking. Because file based locking does not work on file-only shares
For the GA we chose option 2. Therefore we need to remove or disable the file based locking functionality of the CS3org WOPI server. The decision was taken because the current file based locking does not work on file-only shares. The current locking also does not guarantee exclusive access to a file since other parts of oCIS like the WebDAV API or other REVA services don't respect the locks.

After the GA we need to implement option 3.

## Pros and Cons of the Options

Expand Down

0 comments on commit ec066b8

Please sign in to comment.