Skip to content
This repository has been archived by the owner on Nov 6, 2021. It is now read-only.

Fix getParameters() limit of 10 parameters #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix getParameters() limit of 10 parameters #32

wants to merge 1 commit into from

Conversation

RishitKedia
Copy link

Hey there! 👋

So, getParameters() has a limit of getting a maximum of 10 parameters only. 😭

It results in the following error if the number of parameters is more than 10:

UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1):
ValidationException: 1 validation error detected: Value '[omitted-parameter-names]' at 'names'
failed to satisfy constraint: Member must have length less than or equal to 10

This PR fixes this by calling getParameters() multiple times depending on the total number of parameters. 🎉

randytarampi added a commit to randytarampi/me that referenced this pull request Oct 31, 2018
@randytarampi
feat(posts): Setup for #79, #178 and randytarampi/me.words#2 by using…
e7d9c3e 
… `RishitKedia/serverless-secrets#fix-get-parameters-limit-of-10`.

Per trek10inc/serverless-secrets#32.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@RishitKedia