Skip to content

Commit 2bd457d

Browse files
authored
Fix script injection (#597)
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
1 parent e75b9bd commit 2bd457d

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

.github/workflows/release-build.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ env:
2323
# Legacy list of commercial regions to deploy to. New regions should NOT be added here, and instead should be added to the `aws_region` default input to the workflow.
2424
LEGACY_COMMERCIAL_REGIONS: us-east-1, us-east-2, us-west-1, us-west-2, ap-south-1, ap-northeast-3, ap-northeast-2, ap-southeast-1, ap-southeast-2, ap-northeast-1, ca-central-1, eu-central-1, eu-west-1, eu-west-2, eu-west-3, eu-north-1, sa-east-1
2525
LAYER_NAME: AWSOpenTelemetryDistroPython
26+
VERSION: ${{ github.event.inputs.version }}
2627

2728
permissions:
2829
id-token: write
@@ -530,9 +531,9 @@ jobs:
530531
if: steps.ecr-signing-profile.outputs.profile_arn != ''
531532
run: |
532533
# Sign the released public ECR image
533-
notation sign ${{ env.RELEASE_PUBLIC_REPOSITORY }}:v${{ github.event.inputs.version }} \
534+
notation sign ${{ env.RELEASE_PUBLIC_REPOSITORY }}:v${{ env.VERSION }} \
534535
--plugin com.amazonaws.signer.notation.plugin \
535536
--id ${{ steps.ecr-signing-profile.outputs.profile_arn }}
536537
echo "Successfully signed public ECR image"
537-
echo "Image: ${{ env.RELEASE_PUBLIC_REPOSITORY }}:v${{ github.event.inputs.version }}"
538+
echo "Image: ${{ env.RELEASE_PUBLIC_REPOSITORY }}:v${{ env.VERSION }}"
538539
echo "Profile ARN: ${{ steps.ecr-signing-profile.outputs.profile_arn }}"

0 commit comments

Comments
 (0)