Open zaq42 opened 1 month ago
Findings:
arn:aws(-[\\w]+)*:(codestar-connections|codeconnections):.+:[0-9]{12}:host\\/.+
(take a note that in some instances, it might use codestar-connections
in ARN).
arn:aws:codestar-connections:us-east-2:<<account-id>>:connection/<<some-guid-value>>
.To update references in documentation, it is not required to change the ARN in the example.
There are also references to CodeStar Connections in Readme.
Also, although this appears to be a documentation only change, we still have codestar-connections
in https://github.com/aws/aws-cdk/tree/main/packages/aws-cdk-lib/aws-codepipeline-actions/lib. CDK should deprecate that as well (at least to offer a new class). Needs discussion with the team.
For now, we should update the Readme and API documentation string to mention about deprecation.
Describe the issue
Documentation for
CodePipelineSource.connection
says:The term "CodeStar connection" is confusing, since AWS CodeStar is now deprecated.
However, what is needed is now under AWS CodeConnections (ref)
Links
https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.pipelines.CodePipelineSource.html#static-connectionrepostring-branch-props