-
Notifications
You must be signed in to change notification settings - Fork 4.8k
New issue
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
docs(aws-lambda): add name format description for aws lambda plugin #12782
Conversation
dfa320e
to
cf5b8dd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See above
Co-authored-by: Aapo Talvensaari <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to be sure this plugin is using lua-resty-aws
which generates it's API based on: aws/aws-sdk-js and here we're using CreateFunctionCommand which as it's name accepts:
The name or ARN of the Lambda function.
Name formatsFunction name – my-function.
Function ARN – arn:aws:lambda:us-west-2:123456789012:function:my-function.
Partial ARN – 123456789012:function:my-function.
The length constraint applies only to the full ARN. If you specify only the function name, it is limited to 64 characters in length.
Am I getting this right?
@nowNick All correct except the API we're using is actually InvokeFunction which invokes a Lambda function. But the function name format is the same, it is the same for all these APIs related to Lambda function. As long as AWS API supports that format we'll support that too |
Successfully created cherry-pick PR for |
Summary
This PR adds a detailed description for the
function_name
format of the aws-lambda plugin.Checklist
changelog/unreleased/kong
orskip-changelog
label added on PR if changelog is unnecessary. README.md