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

[BUG] Image is not refreshed when other client modifies it #3021

Closed
jesmrec opened this issue Nov 24, 2020 · 3 comments · Fixed by #3025
Closed

[BUG] Image is not refreshed when other client modifies it #3021

jesmrec opened this issue Nov 24, 2020 · 3 comments · Fixed by #3025
Labels
Estimation - 2 (S) p2-high Escalation, on top of current planning, release blocker Type:bug
Milestone

Comments

@jesmrec
Copy link
Collaborator

jesmrec commented Nov 24, 2020

Steps to reproduce

  1. Download an image in the Android app
  2. With other client, modify the image
  3. In the app, open the image again

Current behaviour

Image is not updated, you see the old version. Thumbnail is updated.

Expected behaviour

New version of the image in the server, so in the moment of downloading, the new version must be downloaded and displayed.

Environment data

Android version: 11 & 7

Device model: Pixel 2 & Nexus 6P

ownCloud app version: 2.15.3

ownCloud server version: 10.5

@jesmrec jesmrec added Type:bug p2-high Escalation, on top of current planning, release blocker labels Nov 24, 2020
@jesmrec jesmrec added this to the 2.16-current milestone Nov 24, 2020
@abelgardep
Copy link
Contributor

We use glide to show images, and it uses its own cache, so when we update the image, it keeps using its cached image.

@jesmrec
Copy link
Collaborator Author

jesmrec commented Nov 24, 2020

From my pov, this is not a good behaviour, no matter where the problem is. It is not a data loss, but points to a lack of sync. We have to look for posible solutions.

@abelgardep
Copy link
Contributor

Of course it is not a good behaviour. I just pointed the problem. It should be fixed. Options:

  1. Don't let Glide use its cache
  2. Try to tell glide when it needs to refresh its cache

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Estimation - 2 (S) p2-high Escalation, on top of current planning, release blocker Type:bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants