Skip to content
This repository was 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.

1 participant

@RishitKedia