We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
true
-no-
I wanted to create a function using env var AWS_FUNCTION_NAME=header-no-cache but that results in a function name called true.
AWS_FUNCTION_NAME=header-no-cache
I digged in and found that node-lambda uses commander.js like this:
node-lambda
.option('-n, --functionName [' + AWS_FUNCTION_NAME + ']', 'Lambda FunctionName', AWS_FUNCTION_NAME)
That results in flag (the 1st argument for option): -n, --functionName [header-no-cache]
-n, --functionName [header-no-cache]
Notice the name of my function in flag configuration. Instead of header-no-cache there should be something like functionName.
header-no-cache
functionName
The problem is that thanks to -no- string in the flag configuration commander.js thinks that it's a boolean parameter with default value true.
Solution is not to include input from env vars to the flag definition.
... and PR is coming 🤞
The text was updated successfully, but these errors were encountered:
No branches or pull requests
I wanted to create a function using env var
AWS_FUNCTION_NAME=header-no-cache
but that results in a function name calledtrue
.I digged in and found that
node-lambda
uses commander.js like this:That results in flag (the 1st argument for option):
-n, --functionName [header-no-cache]
Notice the name of my function in flag configuration. Instead of
header-no-cache
there should be something likefunctionName
.The problem is that thanks to
-no-
string in the flag configuration commander.js thinks that it's a boolean parameter with default valuetrue
.Solution is not to include input from env vars to the flag definition.
... and PR is coming 🤞
The text was updated successfully, but these errors were encountered: