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

owncloud storage driver doesn't update file path in redis #1693

Closed
C0rby opened this issue May 7, 2021 · 0 comments · Fixed by #1696
Closed

owncloud storage driver doesn't update file path in redis #1693

C0rby opened this issue May 7, 2021 · 0 comments · Fixed by #1696
Assignees

Comments

@C0rby
Copy link
Contributor

C0rby commented May 7, 2021

The ownCloud storage driver stores the file path in redis using the fileid as the key.
When the file gets renamed 'Moved' the filepath doesn't get updated which causes problems when we want to get the path of that file later.

The filepath must be updated on 'Moves' and removed on 'Delete'

@C0rby C0rby self-assigned this May 7, 2021
@C0rby C0rby mentioned this issue May 7, 2021
C0rby pushed a commit to C0rby/reva that referenced this issue May 10, 2021
Update the filepath in redis when moving a file. Didn't implement it in delete since delete is still a bit more broken. But since we don't actively use the owncloud storage driver except for in CI it's not too important.

Fixes cs3org#1693
C0rby pushed a commit to C0rby/reva that referenced this issue May 10, 2021
Update the filepath in redis when moving a file. Didn't implement it in delete since delete is still a bit more broken. But since we don't actively use the owncloud storage driver except for in CI it's not too important.

Fixes cs3org#1693
C0rby pushed a commit to C0rby/reva that referenced this issue May 10, 2021
Update the filepath in redis when moving a file. Didn't implement it in delete since delete is still a bit more broken. But since we don't actively use the owncloud storage driver except for in CI it's not too important.

Fixes cs3org#1693
ishank011 pushed a commit that referenced this issue May 10, 2021
Update the filepath in redis when moving a file. Didn't implement it in delete since delete is still a bit more broken. But since we don't actively use the owncloud storage driver except for in CI it's not too important.

Fixes #1693
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant