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

chore: namespace all private packages #16689

Merged
merged 6 commits into from
Sep 29, 2021
Merged

Conversation

RomainMuller
Copy link
Contributor

In order to remove any risk of supply-chain attacks leveraging naming
collisions with any of our private-only packages, move all of them into
the @aws-cdk/ namespace, so that no third party can possibly seize one
of these names.

Also, normalized the order of dependencies in package.json files so
they are consistently alphanumerically ordered.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

In order to remove any risk of supply-chain attacks leveraging naming
collisions with any of our private-only packages, move all of them into
the `@aws-cdk/` namespace, so that no third party can possibly seize one
of these names.

Also, normalized the order of dependencies in `package.json` files so
they are consistently alphanumerically ordered.
@RomainMuller RomainMuller self-assigned this Sep 28, 2021
@gitpod-io
Copy link

gitpod-io bot commented Sep 28, 2021

@RomainMuller RomainMuller requested a review from a team September 28, 2021 12:39
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Sep 28, 2021
@RomainMuller RomainMuller force-pushed the rmuller/private-packages branch 3 times, most recently from 4857d47 to 510daee Compare September 28, 2021 15:42
@RomainMuller RomainMuller force-pushed the rmuller/private-packages branch 2 times, most recently from 19abc19 to d9ffa77 Compare September 28, 2021 16:17
…kages

# Conflicts:
#	packages/@aws-cdk/aws-applicationautoscaling/package.json
#	packages/@aws-cdk/aws-chatbot/package.json
#	packages/@aws-cdk/aws-codecommit/package.json
#	packages/@aws-cdk/aws-codepipeline/package.json
#	packages/@aws-cdk/aws-ecr-assets/package.json
#	packages/@aws-cdk/aws-ecr/package.json
#	packages/@aws-cdk/aws-logs/package.json
#	packages/@aws-cdk/aws-s3-assets/package.json
#	packages/@aws-cdk/aws-s3/package.json
#	packages/@aws-cdk/aws-sqs/package.json
#	packages/@aws-cdk/aws-ssm/package.json
@RomainMuller RomainMuller force-pushed the rmuller/private-packages branch from d9ffa77 to de8a77d Compare September 28, 2021 16:47
@RomainMuller RomainMuller force-pushed the rmuller/private-packages branch from 3bdd3d0 to d109026 Compare September 29, 2021 10:47
@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
  • Commit ID: a3577aa
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@RomainMuller RomainMuller merged commit 2f729c4 into master Sep 29, 2021
@RomainMuller RomainMuller deleted the rmuller/private-packages branch September 29, 2021 12:24
TikiTDO pushed a commit to TikiTDO/aws-cdk that referenced this pull request Feb 21, 2022
In order to remove any risk of supply-chain attacks leveraging naming
collisions with any of our private-only packages, move all of them into
the `@aws-cdk/` namespace, so that no third party can possibly seize one
of these names.

Also, normalized the order of dependencies in `package.json` files so
they are consistently alphanumerically ordered.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution/core This is a PR that came from AWS.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants