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

S3 Objectstore as Primary Storage, Files API malfunction. #24417

Closed
hahagu opened this issue Nov 27, 2020 · 2 comments · Fixed by #27561
Closed

S3 Objectstore as Primary Storage, Files API malfunction. #24417

hahagu opened this issue Nov 27, 2020 · 2 comments · Fixed by #27561

Comments

@hahagu
Copy link

hahagu commented Nov 27, 2020

When using S3 objectstore as primary storage, apps such as the Photos app and the Recommendations app fail to show new files that have been added to the server.
Considering that this happens in not only a single app but multiple apps, which are all using the Files API, the problem is suspected to be a bug with the API itself, and not the apps.

nextcloud/photos#522
nextcloud/recommendations#337

There is no errors within the browser, or logged error messages on the server side.
It can only be concluded, that the API itself is to fault.
Thanks!

@hahagu hahagu added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Nov 27, 2020
@hahagu
Copy link
Author

hahagu commented Nov 29, 2020

Researching the code of photos and recommendations app,
the malfunction is possibly narrowed down to three functions responsible:

OCP\Files\Folder -> getDirectoryListing();
OCP\Files\Fileinfo ->isShared();
*OCP\Files\Fileinfo ->getMimeType();

(* means unlikely, as mime types are observed to be working fine from my own testing.)

@szaimen
Copy link
Contributor

szaimen commented Jun 22, 2021

This might get fixed with #27561, too

@szaimen szaimen added 2. developing Work in progress and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jun 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants