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

[ aws-xray-sdk-fastify] Typo of FastityLoggerInstance AND aws-sdk dependency absent #587

Closed
vaibhavgupta-bst opened this issue Apr 26, 2023 · 1 comment
Assignees

Comments

@vaibhavgupta-bst
Copy link

After installing aws-xray-sdk-fastify^3.5.0 and using it as plugin, I got a few errors and below are my queries for the same

  1. There's no attribute FastityLoggerInstance in fastify instead it should have been FastifyLoggerInstance [ aws-xray-sdk-fastify/lib/plugin.d.ts ]?
    image

  2. aws-sdk dependency is not present [ aws-xray-sdk-fastify/lib/private/configure-aws-x-ray-sync.js ]
    image

@carolabadeer
Copy link
Contributor

Hi @vaibhavgupta-bst, thank you for creating this issue and bringing this to our attention! I have created a PR (#589) to fix the typo you pointed out in FastifyLoggerInstance.

Regarding the second issue you mentioned, we take a direct dependency on the aws-sdk package in the root package.json, but not in the sub-packages contained within this SDK. Since the direct dependency is in the root package.json, all packages contained within this monorepo will pull their dependencies from there and add on the dependencies in their respective package.json. You'll notice that's also the case for the aws-xray-sdk-core package, which contains all the AWS SDK and HTTP instrumentation logic (AWSXRay.captureAWS function).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants