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

No password length constraint #3871

Closed
munteanstefan opened this issue May 30, 2024 · 2 comments
Closed

No password length constraint #3871

munteanstefan opened this issue May 30, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@munteanstefan
Copy link

Describe the issue

No password length constraint may lead blocking of the login functionality, and it cached a response:
{"message":"Request Entity Too Large","_links":{"self":{"href":"/ui/admin/stats"}},"_embedded":{"errors":[{"message":"Request Entity Too Large"}]}}

Preconditions:
kestra started up and at least one flow is saved in dashboard

Steps to reproduce

1 Activate basic authorization
2 insert a huge string in the email/password/confirm password field example : input a string of 2997 characters
3 Save the credentials and try to login with the correct email and password

Note that the response is cached, so opening a new tab will result in the same response, check screenshot
image

Environment

  • Kestra Version:
  • Operating System (OS/Docker/Kubernetes):
  • Java Version (if you don't run kestra in Docker):
@munteanstefan munteanstefan added the bug Something isn't working label May 30, 2024
@loicmathieu loicmathieu added this to the v0.18.0 milestone May 30, 2024
@anna-geller anna-geller removed this from the v0.18.0 milestone Jun 16, 2024
@frankzengjj
Copy link
Contributor

hi I would like to take this issue. The PR is here.

@MilosPaunovic
Copy link
Member

Closed by #4261. Thanks @frankzengjj for the contribution! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

5 participants