Skip to content

[stable26] Avoid cleanup on close to not run into race conditions with other sessions on the same document #3922

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.

Sign up for GitHub

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

Merged
juliusknorr merged 10 commits into from
Mar 9, 2023
Merged

[stable26] Avoid cleanup on close to not run into race conditions with other sessions on the same document #3922

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
fcb3ce8
feat(logging): Add debug logging for empty file saving
juliusknorr Mar 7, 2023
2dd921d
fix: Rework save locking
juliusknorr Mar 7, 2023
a5b9229
feat(logging): Add some logging to document cleanup
juliusknorr Mar 7, 2023
9754f1f
fix: Do not cleanup on close
juliusknorr Mar 7, 2023
4eac11d
fix: Cleanup attachments on itinial document cleanup
juliusknorr Mar 7, 2023
949b404
fix: Still unlock documents on close
juliusknorr Mar 8, 2023
dba594d
chore: Make resetDocument more readable
juliusknorr Mar 8, 2023
45662b5
fix: Do not reset on create
juliusknorr Mar 8, 2023
a3c6fb8
fix: Cleanup actual documents in cron
juliusknorr Mar 8, 2023
b38032e
ci: Adapt cypress tests to new cleanup behaviour
juliusknorr Mar 8, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
chore: Make resetDocument more readable 
Signed-off-by: Julius Härtl <jus@bitgrid.net>
  • Loading branch information
@juliusknorr @backportbot-nextcloud
juliusknorr authored and backportbot-nextcloud[bot] committed Mar 9, 2023
commit dba594d564d4485c84aabe8ca1ac452865ba2b9c
30 changes: 13 additions & 17 deletions lib/Service/DocumentService.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -375,30 +375,26 @@ public function autosave(?File $file, int $documentId, int $version, ?string $au
}

/**
* @param $documentId
* @param bool $force
* @throws DocumentHasUnsavedChangesException
* @throws Exception
* @throws NotPermittedException
*/
public function resetDocument(int $documentId, bool $force = false): void {
try {
$this->unlock($documentId);

$document = $this->documentMapper->find($documentId);

if ($force || !$this->hasUnsavedChanges($document)) {
$this->stepMapper->deleteAll($documentId);
$this->sessionMapper->deleteByDocumentId($documentId);
$this->documentMapper->delete($document);

try {
$this->getStateFile($documentId)->delete();
} catch (NotFoundException $e) {
} catch (NotPermittedException $e) {
}
} elseif ($this->hasUnsavedChanges($document)) {
if (!$force && $this->hasUnsavedChanges($document)) {
throw new DocumentHasUnsavedChangesException('Did not reset document, as it has unsaved changes');
}
} catch (DoesNotExistException $e) {

$this->unlock($documentId);

$this->stepMapper->deleteAll($documentId);
$this->sessionMapper->deleteByDocumentId($documentId);
$this->documentMapper->delete($document);

$this->getStateFile($documentId)->delete();
} catch (DoesNotExistException|NotFoundException $e) {
// Ignore if document not found or state file not found
}
}

Expand Down