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

[MacCoss] Issue 50841: Include detail when basic auth password matches but does not meet complexity rules #5692

Open
4 tasks done
labkey-jeckels opened this issue Jul 17, 2024 · 1 comment · Fixed by #5683
Assignees

Comments

@labkey-jeckels
Copy link
Contributor

labkey-jeckels commented Jul 17, 2024

Issue 50841

  • Dev work @labkey-adam
  • Fix automated tests @labkey-adam
  • Test Python client API @labkey-nicka
  • Test R client API @labkey-klum
    • Valid password which doesn't meet complexity requirements. Server log shows : failed to login: password does not meet the complexity requirements
    • However the client is just seeing this message "exception" : "User does not have permission to perform this operation.". Perhaps still throwing UnauthorizedException?

Test scenario:

  • Valid credentials
  • Bad credentials
  • Credentials that match the stored password but are expired
    -- Note the Database Authentication "expire every five seconds" option that makes testing this easy
  • Credentials that match the stored password but don't meet current complexity requirements
    -- Reduce complexity requirement
    -- Create user and set minimal password
    -- Raise complexity requirement
    -- Make API call with minimal credentials
    -- Verify response includes message about complexity rule and needing to change password
@labkey-nicka
Copy link
Contributor

The python API processes responses based on HTTP status codes so it handles these cases as I'd expect. Verified valid credentials, bad credentials, stored password expires, and match but don't meet complexity requirements. All returned the message I expected.

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

Successfully merging a pull request may close this issue.

3 participants