Skip to content
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

Add support for x-ray #16

Open
jasonwilson opened this issue Jul 30, 2019 · 0 comments
Open

Add support for x-ray #16

jasonwilson opened this issue Jul 30, 2019 · 0 comments
Labels
enhancement New feature or request

Comments

@jasonwilson
Copy link

jasonwilson commented Jul 30, 2019

Is your feature request related to a problem? Please describe.
This is a great showcase of an e2e demo application built upon AWS resources. It would be even better if it was instrumented with x-ray out of the box.

Describe the solution you'd like

  • Add AWS instrumentation for Node/Python lambdas using http capture API.
  • Enable x-ray tracing by default on all resources (e.g: api gateway, lambda, etc)

Describe alternatives you've considered
I've gone through and done some of this work myself but it's time consuming given the cloudformation setup uses prebuilt assets from s3. This pain could be reduced if the current setup used CodePipeline to build and deploy the lambda artifacts on demand.

Also worth noting the redis client used in Node.js doesn't support cls under the hood. Would require migrating to ioredis. This may be mitigated by cls_hooked

Additional context
N/A

@triggan triggan added the enhancement New feature or request label Aug 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants