amplify-hosting icon indicating copy to clipboard operation
amplify-hosting copied to clipboard

Access Control not accepting password

Open jasondavis87 opened this issue 3 years ago • 1 comments

Before opening, please confirm:

App Id

No response

Region

us-east-1

Amplify Hosting feature

Access control

Describe the bug

Hey everyone, i dont believe this is my error, but i tried to secure a couple of my branches and the amplify console was not taking the updates. i was using:

  • username: test
  • password: &k$:{fmy^bbw'j2

I would go Access control --> Manage access --> Change the Access setting to Restricted - password required --> Add the username and password twice Upon doing this, i got green messages saying that every other branch was updated other than the one i was trying to change as well as the change never took effect.

I eventially got it to work by using:

  • password: testpassword (which i've since changed to something else alpha numeric which also works)

My guess is one of the special characters in my initial password wasnt escaped somewhere so it was causing an error. My Chrome Console also showed this:

Screenshot 2022-09-14 at 1 28 43 PM

Expected behavior

I expected a password with symbols to work

Reproduction steps

  1. Go into the Amplify App
  2. Goto App settings -> Access Control
  3. Hit the Manage access button
  4. change the Access setting for a branch to 'Restricted - password required'
  5. Enter a username and password
  6. Hit Save

Build Settings

No response

Additional information

No response

jasondavis87 avatar Sep 14 '22 17:09 jasondavis87

Hi @jasondavis87 👋🏽 thanks for raising this issue. I was able to reproduce this behavior and root cause the issue. You can't create passwords that have a colon : in them but the other characters you mentioned are acceptable.

Passwords created with : will not be saved and no warning message occurs either which is not ideal user experience. I have added this bug to our product backlog. We will update this issue once a fix has been pushed. Thank you!

ghost avatar Sep 15 '22 17:09 ghost