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

Public link, enforce password broken #12272

Closed
schiessle opened this issue Nov 5, 2018 · 3 comments
Closed

Public link, enforce password broken #12272

schiessle opened this issue Nov 5, 2018 · 3 comments
Assignees
Labels
Milestone

Comments

@schiessle
Copy link
Member

I just noticed that the "enforce password" setting for link shares is broken on master.

  1. Enable "enforce password" in the admin share settings
  2. Try to create a public link
  3. On first click nothing happens, also no error messages in the JS console or nextcloud.log
  4. Click again on the "+" I get a banner at the top "unable to create a link share", again no additional error message in the logs and the share dialog looks like this:

image

@skjnldsv could you have a look? Thanks!

@skjnldsv
Copy link
Member

skjnldsv commented Nov 5, 2018

@schiessle Hum, this is properly working here.
Can you make sure your cache is cleared?

@schiessle
Copy link
Member Author

@skjnldsv I was confused as well, because I knew that you and @rullzer tested it before we merged the pr. But I can reliable reproduce it here. It is a fresh installation (current master) also opened a private window to make sure no old data are cached.

I tried it with two browsers:

Firefox 63.0.1
Chromium 70.0.3538.67

@skjnldsv
Copy link
Member

skjnldsv commented Nov 5, 2018

yep, something else broke it!
Let me handle this

@skjnldsv skjnldsv self-assigned this Nov 5, 2018
@skjnldsv skjnldsv added the 2. developing Work in progress label Nov 5, 2018
@nextcloud nextcloud deleted a comment from nextcloud-bot Nov 5, 2018
@skjnldsv skjnldsv added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Nov 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants
@schiessle @skjnldsv and others