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

OCS user creation: bad request when password policy not passed #7946

Closed
devnix opened this issue Jan 18, 2018 · 3 comments · Fixed by #8502
Closed

OCS user creation: bad request when password policy not passed #7946

devnix opened this issue Jan 18, 2018 · 3 comments · Fixed by #8502
Labels
3. to review Waiting for reviews bug
Milestone

Comments

@devnix
Copy link
Member

devnix commented Jan 18, 2018

After a good amount of time scratching my head, I think I found a bug (or, at least, a very misinformative error):

When I try to create a new user using the OCS API and the password does not meet the system's password policy, the server will return a "Bad request" message:

{"ocs"=>{"meta"=>{"status"=>"failure", "statuscode"=>101, "message"=>"Bad request", "totalitems"=>"", "itemsperpage"=>""}, "data"=>[]}}
@MorrisJobke
Copy link
Member

Correct. I found the issue and will fix it.

MorrisJobke added a commit that referenced this issue Feb 22, 2018
* adds a 107 error code together with the hint of the exception
* logs the exception as warning
* fixes #7946

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke
Copy link
Member

Fix is in #8502

@MorrisJobke MorrisJobke added bug 3. to review Waiting for reviews labels Feb 22, 2018
MorrisJobke added a commit that referenced this issue Feb 23, 2018
* adds a 107 error code together with the hint of the exception
* logs the exception as warning
* fixes #7946

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
MorrisJobke added a commit that referenced this issue Feb 23, 2018
* adds a 107 error code together with the hint of the exception
* logs the exception as warning
* fixes #7946

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke
Copy link
Member

It's very likely that this will be also fixed in 12.0.6 and 13.0.1 ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants