Skip to content

Commit c74a2af

Browse files
[Lambda] Extract input from workflow as environment variable (#137)
Quick fix in reference to an input injection security issue detected automatically. This will need to be cherry picked/PRed to other branches as well, namely the release branches 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 fec9390 commit c74a2af

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

.github/workflows/release-lambda.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ on:
99
default: '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, af-south-1, ap-east-1, ap-south-2, ap-southeast-3, ap-southeast-4, eu-central-2, eu-south-1, eu-south-2, il-central-1, me-central-1, me-south-1'
1010

1111
env:
12+
AWS_REGIONS: ${{ github.event.inputs.aws_region }}
1213
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
1314
LAYER_NAME: AWSOpenTelemetryDistroJs
1415

@@ -25,7 +26,7 @@ jobs:
2526
- name: Set up regions matrix
2627
id: set-matrix
2728
run: |
28-
IFS=',' read -ra REGIONS <<< "${{ github.event.inputs.aws_region }}"
29+
IFS=',' read -ra REGIONS <<< "${{ env.AWS_REGIONS }}"
2930
MATRIX="["
3031
for region in "${REGIONS[@]}"; do
3132
trimmed_region=$(echo "$region" | xargs)

0 commit comments

Comments
 (0)