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

"Create and copy public link to clipboard" generates an invalid link #4380

Closed
algihu opened this issue Aug 10, 2022 · 11 comments
Closed

"Create and copy public link to clipboard" generates an invalid link #4380

algihu opened this issue Aug 10, 2022 · 11 comments

Comments

@algihu
Copy link

algihu commented Aug 10, 2022

"Create and copy public link to clipboard" generates an invalid link

Describe the bug

Using the Windows desktop app pointed to an OCIS instance, when I right-click on a synced file in Windows and select "Create and copy public link to clipboard" a link is generated and copied to the clipboard but when browsing to that link an "Unknown error" is returned.

Steps to reproduce

Steps to reproduce the behavior:

  1. Set up ownCloud Desktop App (Windows) pointed to an OCIS server
  2. In Windows Explorer navigate to any file in ownCloud and right-click on it
  3. In the context menu, select ownCloud-->Create and copy public link to clipboard
  4. Open a web browser (I used Chrome) and paste/navigate to the URL that was copied to your clipboard.

Expected behavior

The browser should load a page where you can view/download the file

Actual behavior

Although a link gets generated and copied to the clipboard, when I view the file's properties I can see that no link was actually associated with this file and when I try to navigate to the URL a page is show that says "Unknown error"
image

Here is a video showing this behavior:
Sharing Not Working

Setup

Installed/Started OCIS 2.0.0-beta.5 using the quick start guide here: https://doc.owncloud.com/ocis/next/quickguide/quickguide.html

Installed ownCloud Desktop App 2.10.1 (build 7187) and configured it to sync as virtual files to my OCIS instance.

Created a folder with some pictures in it.

Additional context

I can generate a valid link using the dialog that opens when I select ownCloud-->Share. After clicking that option I select the "Public Links" tab, click on the "Create new..." option and then click the "Create" button. I then click the "..." next to the newly created Public Link and select "Copy Link to Clipboard" and then paste/navigate to that link in a browser, This successfully allows me to access the file.

Here is a video showing this working successfully.
Sharing Working

@michl19
Copy link
Contributor

michl19 commented Aug 22, 2022

@michaelstingl can you have a look into this as well. We're unsure if it's desktop or backend.

@michaelstingl
Copy link
Contributor

We're unsure if it's desktop or backend.

Maybe desktop. Something's 🐟 -y. I'll have a look tomorrow…

@kulmann
Copy link
Member

kulmann commented Aug 23, 2022

What kind of permissions does the public link have? We've introduced so called Internal links, which have a permission bitmask of 0. Those are supposed to resolve into the correct shared location in the authenticated context of the user.

If the created link has permissions 0 then I'd have an explanation for the issue described above: Resolving internal links is not yet supported in ownCloud Web - support is coming with owncloud/web#7405 which is close to being merged.

If it's a link with permissions > 0 then I have no idea, yet.

@michaelstingl
Copy link
Contributor

michaelstingl commented Aug 23, 2022

Because of #4445 , Desktop sync client 2.11 tries to create a link with todays expiration date, and because of #4446 , the link creation fails silently.

@michaelstingl
Copy link
Contributor

No idea why both ways behave different in the desktop sync client. I opened an issue for further investigation: owncloud/client#10057

@C0rby
Copy link
Contributor

C0rby commented Sep 14, 2022

@michaelstingl, can you still reproduce this? I can't but I'm testing it on Linux since I don't have Windows.

@kulmann
Copy link
Member

kulmann commented Sep 14, 2022

Should have been fixed with #4475

@C0rby
Copy link
Contributor

C0rby commented Sep 14, 2022

Cool so I can close this issue?

@kulmann
Copy link
Member

kulmann commented Sep 14, 2022

yes 😁 @algihu please re-open if the issue is not fixed for you with ocis v2.0.0-beta.8

@C0rby C0rby closed this as completed Sep 14, 2022
@algihu
Copy link
Author

algihu commented Sep 15, 2022

I just tested it again with v2.0.0-beta.8 and this feature now works correctly.

@kulmann
Copy link
Member

kulmann commented Sep 16, 2022

I just tested it again with v2.0.0-beta.8 and this feature now works correctly.

Thanks for investing the time ❤️ Glad to hear that it's fixed 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

6 participants