-
Notifications
You must be signed in to change notification settings - Fork 421
Docs: injected Lambda context keys do not match documentation #1828
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
Comments
Thanks for opening your first issue here! We'll come back to you as soon as we can. |
Hi @aidansteele thank you so much for catching this! The documentation is wrong, I can see that the keys never had the |
@rubenfonseca thanks for confirming. Sure, I'll submit a PR ASAP 👍 |
|
This is now released under 2.6.0 version! |
What were you searching in the docs?
The
inject_lambda_context_output.json
example says the following keys should be emitted:The actual keys (as of version 2.5.0, layer version 17) that get emitted are:
Specifically: the
lambda_
prefix is dropped fromlambda_function_name
,lambda_function_memory_size
,lambda_function_arn
and thelambda_request_id
is renamed tofunction_request_id
.Is this related to an existing documentation section?
https://awslabs.github.io/aws-lambda-powertools-python/2.5.0/core/logger/#capturing-lambda-context-info
How can we improve?
The examples should match the output generated by the library.
Got a suggestion in mind?
It's not clear if this is a bug in the docs or the code. This GitHub search results page shows where the incorrect keys appear in the docs - I suspect it's easier to fix the docs? But I'll defer to the maintainer's judgment.
Acknowledgment
The text was updated successfully, but these errors were encountered: