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

Weird behaviour after trying to share a file via link with a password that not fit with password policy pattern #4209

Closed
davitol opened this issue Nov 27, 2015 · 13 comments

Comments

@davitol
Copy link

davitol commented Nov 27, 2015

Desktop Client version: Version 2.1.0-nightly20151127 (build 2916) (OSX)

Steps to reproduce:

  1. Share via link a file
  2. Try to set a password that does not fit with password policy pattern

Expected behaviour

A message should be shown warning about password policy

Actual behaviour

The spinner keeps loading and the desktop client turns into a weird behaviour

logs:
{"status":"failure","statuscode":403,"message":"Password is too short. Minimum 6 characters are required."},"data":[]}}"

screen shot 2015-11-27 at 11 33 46

@davitol davitol added this to the 2.1-current milestone Nov 27, 2015
@davitol davitol changed the title Weird behaviour after trying to share a file via link with a password that not fit with password policy patter Weird behaviour after trying to share a file via link with a password that not fit with password policy pattern Nov 27, 2015
@dragotin
Copy link
Contributor

Well, I think this is a rather minor problem and can be moved to the next release.

@dragotin dragotin modified the milestones: 2.1.1-nextpatch, 2.1-current Nov 30, 2015
@dragotin dragotin modified the milestones: 2.1.2-next, 2.1.1-current Jan 8, 2016
@ckamm
Copy link
Contributor

ckamm commented Jan 19, 2016

@davitol Which password policy app were you using? I tried https://apps.owncloud.com/content/show.php/Password+Policy+Enforcement?content=167207 but that didn't seem to apply the policy to link-share passwords.

Edit: Ah, probably the enterprise one.

@ckamm ckamm self-assigned this Jan 19, 2016
@ckamm ckamm added the ReadyToTest QA, please validate the fix/enhancement label Jan 20, 2016
@davitol
Copy link
Author

davitol commented Jan 22, 2016

@ckamm Tested again with Version 2.1.1-nightly20160122 (build 3053) for MacOSX and it is NOT fixed 👎

@ckamm
Copy link
Contributor

ckamm commented Jan 22, 2016

@davitol Thanks for testing, it worked for me, but I'll test again.

@ckamm
Copy link
Contributor

ckamm commented Jan 22, 2016

@davitol It works for me.

  1. I click "Share link"
  2. Since a password is required, keyboard focus jumps to the password dialog. (no spinners are moving at this point)
  3. I enter a password that doesn't fit the policy and hit enter.
  4. An error message appears, telling me that there are too few numbers in the password. Keyboard focus stays on the password field.
  5. I enter a correct password and the share is created.

I think there's a good chance the build you were using didn't include the fix, I'll try to figure that out.

In case it's really unfixed: What password policy settings are you using exactly? How, precisely, can you reproduce it?

@davitol
Copy link
Author

davitol commented Jan 22, 2016

@ckamm I set 8 characters as pwd policy in my 8.2.2 oC server. Then in the client for MacOSX I shared via link and set a pwd of 6 characters. The bug happened and the following log was printed:

{"status":"failure","statuscode":403,"message":"Password is too short. Minimum 8 characters are required."},"data":[]}}"
01-22 10:10:34:037 0x7fee02e0d840 OCC::ShareLinkWidget::displayError: Error from server 403 "Password is too short. Minimum 8 characters are required."

Version 2.1.1-nightly20160122 (build 3053) for MacOSX

@ckamm
Copy link
Contributor

ckamm commented Jan 22, 2016

@danimo verified that the build you are using does indeed include my change. :/

@davitol
Copy link
Author

davitol commented Jan 22, 2016

@ckamm also tested in client Version 2.1.1-nightly20160122 (build 5791) for Windows and the bug also happens :-\

@ckamm
Copy link
Contributor

ckamm commented Jan 22, 2016

@davitol Could you explain again how to reproduce the problem? I'm also using the >=8 character password policy.

@ckamm ckamm removed the ReadyToTest QA, please validate the fix/enhancement label Jan 22, 2016
@ckamm
Copy link
Contributor

ckamm commented Jan 22, 2016

We talked on IRC: The problem is fixed when passwords are required but still fails when passwords are optional.

ckamm added a commit to ckamm/owncloud-client that referenced this issue Jan 22, 2016
@davitol
Copy link
Author

davitol commented Jan 22, 2016

@ckamm Even when pwd is mandatory, there are other scenarios that fail.

  1. Set a pwd that fit with the pwd policy
  2. Check Allow editing & Set expiration date checkboxes
  3. Modify again the pwd but now with one that doesn't match with the pwd policy pattern

The bug is reproduced

@ckamm
Copy link
Contributor

ckamm commented Jan 22, 2016

@davitol The next nightly should have the additional fixes.

@ckamm ckamm added the ReadyToTest QA, please validate the fix/enhancement label Jan 22, 2016
@Dianafg76 Dianafg76 added approved by QA and removed ReadyToTest QA, please validate the fix/enhancement labels Jan 25, 2016
@Dianafg76
Copy link

I tested this issue and now is working OK
Desktop v ownCloud-2.1.1.3067-nightly20160125.pkg
Server v 8.2.2
@davitol Thanks!
@ckamm Thanks!

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

No branches or pull requests

4 participants