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

Redirect / resolving of some id-only based URL schemes broken #9269

Closed
michaelstingl opened this issue Jun 16, 2023 · 10 comments · Fixed by #10154
Closed

Redirect / resolving of some id-only based URL schemes broken #9269

michaelstingl opened this issue Jun 16, 2023 · 10 comments · Fixed by #10154
Assignees
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working

Comments

@michaelstingl
Copy link
Contributor

michaelstingl commented Jun 16, 2023

Steps to reproduce

  1. Login einstein:relativity@ocis.ocis-traefik.latest.owncloud.works
  2. Upload folder Hacker-Songs in Personal Space
  3. Open Hacker-Songs
  4. Copy browser URL
    https://ocis.ocis-traefik.latest.owncloud.works/files/spaces/personal/einstein/Hacker-Songs?fileId=96fd1fa6-2d2a-4701-be6e-7d15a82ecd95%244c510ada-c86b-4815-8820-42cdf82c3d51%2101275b1d-f963-497e-9e76-7a46cf71d72b&items-per-page=100&view-mode=resource-table&tiles-size=1
  5. Id-only based URL as described in ADR-11 doesn't work
    https://ocis.ocis-traefik.latest.owncloud.works/files?fileId=96fd1fa6-2d2a-4701-be6e-7d15a82ecd95%244c510ada-c86b-4815-8820-42cdf82c3d51%2101275b1d-f963-497e-9e76-7a46cf71d72b ➡️ Opens in Personal Space ⚠️
  6. Also https://ocis.ocis-traefik.latest.owncloud.works/files/spaces?fileId=96fd1fa6-2d2a-4701-be6e-7d15a82ecd95%244c510ada-c86b-4815-8820-42cdf82c3d51%2101275b1d-f963-497e-9e76-7a46cf71d72b ➡️ Opens in Personal Space ⚠️
  7. But https://ocis.ocis-traefik.latest.owncloud.works/files/spaces/personal?fileId=96fd1fa6-2d2a-4701-be6e-7d15a82ecd95%244c510ada-c86b-4815-8820-42cdf82c3d51%2101275b1d-f963-497e-9e76-7a46cf71d72b ➡️ Opens in Hacker-Songs in Personal Space ✅
  8. Also oC10-style private link https://ocis.ocis-traefik.latest.owncloud.works/f/96fd1fa6-2d2a-4701-be6e-7d15a82ecd95$4c510ada-c86b-4815-8820-42cdf82c3d51%2101275b1d-f963-497e-9e76-7a46cf71d72b ➡️ Opens fine in Hacker-Songs in Personal Space ✅

Expected behaviour

All id-only based URLs should properly open.

Actual behaviour

Some id-only based URLs doesn't open properly.

Environment ownCloud Infinite Scale

Version:

[Log]  ownCloud Web UI 7.0.1  (index.html-d302d856.mjs, line 7)
[Log]  Infinite Scale 3.1.0-next.1+01c936eae Community  (index.html-d302d856.mjs, line 7)

Client configuration

Browser:
Safari 16.5

Operating system:
macOS Ventura 13.4

@kulmann

@michaelstingl
Copy link
Contributor Author

@micbar @tbsbdr how does the URL in an OCIS activity mail notification look like?

@micbar
Copy link
Contributor

micbar commented Jun 20, 2023

Share

Hello Marie Skłodowska Curie

Admin has shared "Neuer Ordner" with you.

Click here to view it: https://ocis.ocis-wopi.latest.owncloud.works/files/shares/with-me


---
ownCloud - Store. Share. Work.
https://owncloud.com/

Space

Hello Marie Skłodowska Curie,

Admin has invited you to join "Mars".

Click here to view it: https://ocis.ocis-wopi.latest.owncloud.works/f/e8bb1ac8-0fdd-413b-a6ed-ebc79e057b77$8d424a4e-df7a-4e6a-9ffd-8a61d96aa212


---
ownCloud - Store. Share. Work.
https://owncloud.com/

@JammingBen
Copy link
Contributor

Id-only based URLs look like this: https://<host>/f/<resource_id>. I don't not why it differs from the ADR specified in https://owncloud.dev/ocis/adr/0011-global-url-format/#configurable-path-component-in-urls, however, I'm not aware of any plans to change that in the future. (Note that the wording in this section of the ADR is "could" and "would", so it sounds like these are just suggestions?). @michaelstingl does that work for you?

@michaelstingl
Copy link
Contributor Author

@michaelstingl does that work for you?

PO / PM decision @tbsbdr

@micbar
Copy link
Contributor

micbar commented Jun 26, 2023

@JammingBen @michaelstingl
I was under the impression that
This URL https://ocis.ocis-wopi.latest.owncloud.works/files/spaces/project/neuer-space/Neuer%20Ordner?fileId=3dcbbe99-aaab-4fd7-96cf-fd4422a2847d%2434d61e97-cbbd-492c-8f20-3a90db4b2bf8%212942948a-6e18-4edb-8d38-03104994376c&sort-by=name&sort-dir=asc&scrollTo=Neue%20Datei.txt&tiles-size=2&items-per-page=100&view-mode=resource-table

should work without the path prefix

https://ocis.ocis-wopi.latest.owncloud.works/files/spaces/?fileId=3dcbbe99-aaab-4fd7-96cf-fd4422a2847d%2434d61e97-cbbd-492c-8f20-3a90db4b2bf8%212942948a-6e18-4edb-8d38-03104994376c&sort-by=name&sort-dir=asc&scrollTo=Neue%20Datei.txt&tiles-size=2&items-per-page=100&view-mode=resource-table

@JammingBen
Copy link
Contributor

JammingBen commented Jun 26, 2023

No that doesn't work. I'm only aware of the following 2 ways:

  • https://ocis.ocis-wopi.latest.owncloud.works/f/3dcbbe99-aaab-4fd7-96cf-fd4422a2847d%2434d61e97-cbbd-492c-8f20-3a90db4b2bf8%212942948a-6e18-4edb-8d38-03104994376c (file id only)
  • https://ocis.ocis-wopi.latest.owncloud.works/files/spaces/project?fileId=3dcbbe99-aaab-4fd7-96cf-fd4422a2847d%2434d61e97-cbbd-492c-8f20-3a90db4b2bf8%212942948a-6e18-4edb-8d38-03104994376c (file id in combination with the project namespace)

@micbar
Copy link
Contributor

micbar commented Jun 26, 2023

ah, ok

This seems we need to clean up the ADR

@kulmann
Copy link
Contributor

kulmann commented Jun 26, 2023

... it was intended to support an auto-redirect of id-only to path and id. Missing path and wrong path should be treated the same way: resolve to the correct path and id combination.

Seems like we only support the correction of a wrong path at the moment.

However, the private link /f/fileid has nothing to do with that and should and will always work.

@JammingBen
Copy link
Contributor

Okay, so that means we treat this as a bug or an enhancement...?

@kulmann
Copy link
Contributor

kulmann commented Jul 3, 2023

Okay, so that means we treat this as a bug or an enhancement...?

bug :-)

@JammingBen JammingBen added Type:Bug Something isn't working Priority:p3-medium Normal priority labels Jul 3, 2023
@github-project-automation github-project-automation bot moved this to Qualification in Web Support Board Jul 3, 2023
@JammingBen JammingBen moved this from Qualification to Bugs Prio 3 or less in Web Support Board Jul 3, 2023
@kulmann kulmann moved this from Qualification to Prio 3 or less in Infinite Scale Team Board Oct 27, 2023
@JammingBen JammingBen self-assigned this Nov 27, 2023
@JammingBen JammingBen moved this from Prio 3 or less to In progress in Infinite Scale Team Board Nov 27, 2023
@github-project-automation github-project-automation bot moved this from In progress to Done in Infinite Scale Team Board Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working
Projects
Archived in project
Status: Bugs Prio 3 or less
Development

Successfully merging a pull request may close this issue.

4 participants