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

Auth fails if login contains query params #9749

Closed
3 tasks done
u12206050 opened this issue Nov 12, 2021 · 3 comments
Closed
3 tasks done

Auth fails if login contains query params #9749

u12206050 opened this issue Nov 12, 2021 · 3 comments
Assignees

Comments

@u12206050
Copy link
Contributor

Preflight Checklist

Describe the Bug

When trying to login using SSO this happens when the url looks like:

/admin/login WORKS
/admin/login?anything FAILS

To Reproduce

Login using SSO make sure the is a query param eg. ?test at the end of the url

Errors Shown

{"errors":[{"message":"Invalid user credentials.","extensions":{"code":"INVALID_CREDENTIALS"}}]}

What version of Directus are you using?

9.0.1

What version of Node.js are you using?

16.4.0

What database are you using?

mysql 8

What browser are you using?

chrome

What operating system are you using?

osx

How are you deploying Directus?

GCP

@u12206050
Copy link
Contributor Author

Seems to also be a problem on version 9.0.0

@joselcvarela
Copy link
Member

Hello @u12206050
Are you experiencing this issue with the API request? I am not sure I understand what are you experiencing.

I made this request:

POST http://localhost:8055/auth/login?test
Content-Type: application/json
{"email":"xxxx", "password":"xxxx"}

And seems to work as expected.

@u12206050
Copy link
Contributor Author

Hi, after updating to 9.1.2 it works, and checking back it seems to have been fixed in v9.0.1. Thanks

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants