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

uuid pinned dependency has import issues #4120

Open
bencooper222 opened this issue Jun 7, 2022 · 3 comments
Open

uuid pinned dependency has import issues #4120

bencooper222 opened this issue Jun 7, 2022 · 3 comments
Assignees
Labels
bug This issue is a bug. dependencies This issue is a problem in a dependency. p2 This is a standard priority issue

Comments

@bencooper222
Copy link

Describe the bug

aws-sdk no longer works with certain bundlers.

Trying to bundle aws-sdk using Serverless framework for an AWS lambda is failing with an error.

This was an issue with uuid that was resolved in 8.1.0 (see below link) but the latest version of aws-sdk is pinned to version 8.0.0. uuidjs/uuid#444 (comment)

Please upgrade uuid to resolve this

Expected Behavior

Not to see the an error when bundling aws-sdk

Current Behavior

I saw the below error

  Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './package.json' is not defined by "exports" in /Users/benc/code/convoyinc/repo-checks-service/node_modules/aws-sdk/node_modules/uuid/package.json

Reproduction Steps

This is a downstream library problem, AWS should just pull in the downstream library.

Possible Solution

Pull in this fix uuidjs/uuid#444 (comment)

Additional Information/Context

No response

SDK version used

2.1149.0

Environment details (OS name and version, etc.)

MacOS Monterrey

@bencooper222 bencooper222 added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Jun 7, 2022
@bencooper222 bencooper222 changed the title (short issue description) uuid pinned dependency has import issues Jun 8, 2022
@ajredniwja
Copy link
Member

Hi @bencooper222, the 8.0.0 version was used in #3924, we had problems with later version as mentioned there.

@ajredniwja ajredniwja added dependencies This issue is a problem in a dependency. needs-review This issue/pr needs review from an internal developer. and removed needs-triage This issue or PR still needs to be triaged. labels Jun 13, 2022
@ajredniwja ajredniwja self-assigned this Jun 13, 2022
@bencooper222
Copy link
Author

Gotcha, sounds like the ball is in uuid's court then

@MarkFarmiloe
Copy link

uuid has released v9. Is it worth revisiting this?

@yenfryherrerafeliz yenfryherrerafeliz added the p2 This is a standard priority issue label Mar 23, 2023
@RanVaknin RanVaknin assigned RanVaknin and unassigned ajredniwja Nov 29, 2023
@RanVaknin RanVaknin removed the needs-review This issue/pr needs review from an internal developer. label Nov 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. dependencies This issue is a problem in a dependency. p2 This is a standard priority issue
Projects
None yet
Development

No branches or pull requests

5 participants