search

awslabs / sandbox-accounts-for-events

"Sandbox Accounts for Events" allows to provide multiple, temporary AWS accounts to a number of authenticated users simultaneously via a browser-based GUI.
Apache License 2.0
172 stars 37 forks source link

Error to list users with admin permission #45

Closed eduardomartinsdarede closed 10 months ago

eduardomartinsdarede commented 10 months ago

I can't list users after deploy, I think is something in cognito because I can ADD and Remove new users...

image

Error:

image

eduardomartinsdarede commented 10 months ago

More infos:

image

moellr commented 10 months ago

Issue identified, updating the SDK to v3 for AdminQueries broke due to string conversion issue. This seems to be a known bug in Amplify: https://github.com/aws-amplify/amplify-cli/issues/12735 I am currently applying and testing fixes - if you need a quick fix earlier, you can manually update the AdminQeries Lambda with the code from the link above.

eduardomartinsdarede commented 10 months ago

Thanks for reply!

eduardomartinsdarede commented 10 months ago

@moellr the quickfix who worked is it:

https://github.com/aws-amplify/amplify-cli/compare/dev...marcomilon:amplify-cli:category-auth/fix-limit-not-converted-to-number

philippegabert commented 9 months ago

Hi there,

when will this be implemented in this code/repo? How to update an existing installation with the bug fix? (And not having to change the lambda manually)

Thanks

moellr commented 9 months ago

Implemented now with the latest commit, feel free to merge with your local repo.

krishnarastogi commented 8 months ago

@moellr @eduardomartinsdarede I am still experiencing the same issue. Could you please share how you solved it?

moellr commented 8 months ago

Did you perform a reinstall of the whole project with the latest changes? That should fix it. Alternatively you could manually apply these changes to your existing install: https://github.com/awslabs/sandbox-accounts-for-events/commit/a4e3a0a341c590e8c89fa969d5a3843e3e85b7d3 as a hotfix.