-
Notifications
You must be signed in to change notification settings - Fork 408
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
Combine logging and tracing ? #4
Labels
Comments
Hey Andy - Invert that.
Tracer ultimately runs the handler so it should be the last one —
Decorators run from bottom up
@tracer.capture_lambda_handler
@logger...
def handler(event, context):
...
I’ll add an example in 2 weeks (in the middle of something) — Let me know
if it doesn’t work and I can help
…On Mon, 3 Feb 2020 at 20:34, Andy Thomas ***@***.***> wrote:
Hi
What should you put before the lambda handler to use both logging and
tracing ?
I have this:
@logger_inject_lambda_context
@tracer.capture_lambda_handler
def handler(event, context)
The logging is working - but not the tracing.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<aws-powertools/powertools-lambda#4>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAZPQBGUVO5JPIOFEBRJJB3RBB5WPANCNFSM4KPLUGGQ>
.
|
cc @andyt530 in case you missed the response :) |
Resolving as I haven't heard back from Andy. Created an explicit task to add a Lambda function as an example |
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi
What should you put before the lambda handler to use both logging and tracing ?
I have this:
@logger_inject_lambda_context
@tracer.capture_lambda_handler
def handler(event, context)
The text was updated successfully, but these errors were encountered: