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

[QA] OCM - images (& thumbnails) are completely broken with OCM #10272

Closed
Tracked by #9735
jnweiger opened this issue Oct 9, 2024 · 7 comments
Closed
Tracked by #9735

[QA] OCM - images (& thumbnails) are completely broken with OCM #10272

jnweiger opened this issue Oct 9, 2024 · 7 comments
Assignees
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug

Comments

@jnweiger
Copy link
Contributor

jnweiger commented Oct 9, 2024

Describe the bug

A user who received a shared folder via federation cannot view images. The image viewer hangs

Steps to reproduce

Setup two federated machines with e.g.

env OCIS_DNSNAME=dog-DATE OCIS_VERSION=daily ./deploy_ocis_bare_metal.sh
env OCIS_DNSNAME=cat-DATE OCIS_VERSION=daily ./deploy_ocis_bare_metal.sh
ocm_trust.sh dog-20241015.jw-qa.owncloud.works cat-20241015.jw-qa.owncloud.works

  • admin adds the instances to each other's ocmproviders.json .
  • user alice at cat visits the sciencemesh app and create an invite token for user bob at dog.
  • user bob at dog enters the token, and shares a folder smallpics (continaing some png and jpg images) with alice ad cat
  • user alice clicks Shares in the left sidebar, and clicks smallpics ( see [QA] OCM - a federated folder randomly shows 'enable sync' message #10271 )
  • user alice sees no thumbnails in the folder view: BAD

image

  • user alice clicks on one of the images. It does not disaplay in the viewer: BAD

image

  • user alice tries to download an image. A login prompt appears: BAD

image

  • after entering the correct credentials there, the download works. OKish

  • Image viewer still does not. BAD

  • txt and md files work perfectly fine.

Expected behavior

  • thumbnails appear for federated folders too,
  • images can be viewed and downloaded.

Seen with both ocis-6.5.0 and daily 2024-10-09

@JammingBen
Copy link
Contributor

JammingBen commented Oct 11, 2024

There are 2 underlying issues:

  • When sharing a single image file, the file.mimetype property of the driveItem (returned by the server when querying graph/v1beta1/me/drive/sharedWithMe) is application/octet-stream. So Web is not recognizing the file as an image.

  • When sharing a folder that includes an image, the GET request to the image fails with a 404. It looks correct to me though: https://localhost:9200/remote.php/dav/spaces/{remoteItemId}/image.jpg?scalingup=0&preview=1&a=1&processor=thumbnail&c=6f3a16540d82ced6ea769e84aa0a1bd5&x=36&y=36

IMO this is a p2, images are completely broken in OCM. This does not only affect thumbnails, but also viewing images via the preview app. @micbar agree?

@JammingBen JammingBen changed the title [QA] OCM - no thumbnails for images in a received folder, image viewer hangs [QA] OCM - images (& thumbnails) are completely broken with OCM Oct 11, 2024
@JammingBen JammingBen moved this from Qualification to Prio 2 in Infinite Scale Team Board Oct 11, 2024
@JammingBen JammingBen added the Priority:p2-high Escalation, on top of current planning, release blocker label Oct 11, 2024
@micbar
Copy link
Contributor

micbar commented Oct 21, 2024

Scope

  • Previews and Thumbnails can be broken -> Known Issue
  • Downloads and Archiver need to work.

@ScharfViktor
Copy link
Contributor

ScharfViktor commented Oct 25, 2024

@micbar
Copy link
Contributor

micbar commented Oct 25, 2024

@butonic @kwdrago @kobergj @rhafer
After talking today, we came to the conclusion that this is fixable in the scope of the 7.0.0 release. OCM has a storageprovider implemented with a fixed providerID, so the thumbnail and archiver services should be able to use the default implementation by doing an InitiateFileDownloadrequest to the gateway. We have a bug in the implementation maybe, but no "missing" piece.

@rhafer
Copy link
Contributor

rhafer commented Oct 30, 2024

After talking today, we came to the conclusion that this is fixable in the scope of the 7.0.0 release.

If you think so ...

{"level":"error","service":"thumbnails","time":"2024-10-30T15:37:01Z","line":"github.com/owncloud/ocis/v2/services/thumbnails/pkg/service/grpc/v0/service.go:311","message":"resource info is missing checksum"}

The first issue seems to be that the ocm storage provider does not return checksums in its StatResponse. Still trying to figure out why exactly that is...

@rhafer
Copy link
Contributor

rhafer commented Nov 5, 2024

Fix was merged.

@jnweiger
Copy link
Contributor Author

jnweiger commented Dec 3, 2024

Confirmed fixed in 7.0.0-rc.3 + web 11.0.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

5 participants