fix: remove error STRING_VALUE cannot be converted to an Integer (#12… #13157
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of changes
Issue:
When calling the AdminQueries lambda function with the limit parameter, it returns a 500 error because the limit parameter cannot be converted to an integer.
Fix:
Update the code used to generate the AdminQueries lambda function by parsing the limit parameter to return an integer.
Change in this PR:
Added
parseInt
topackages/amplify-category-auth/resources/adminAuth/admin-auth-cognitoActions.js
when using the Limit parameter.Issue #, if available
Description of how you validated changes
I created a sample app and tested the following paths on the AdminQueries lambda function using the limit parameter:
They work as expected after applying my fix return the correct response with http code 200.
Checklist
yarn test
passesBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.