Merge pull request #26 from LeadingEDJE/connect-to-s3-5

split up variables
LeadingEDJE · Nov 10, 2023 · fd96aba · fd96aba
2 parents 9839221 + 6d8bdc8
commit fd96aba
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/.github/workflows/aws.yml b/.github/workflows/aws.yml
@@ -5,7 +5,6 @@ on:
     branches:
       - 'main'
 env:
-  aws_role: ${{ secrets.AWS_ROLE }}
   s3_bucket: ${{ secrets.S3_BUCKET }}
 
 jobs:
@@ -29,11 +28,11 @@ jobs:
       - name: Test
         run: npm run test:headless
       - name: "ECHO AWS"
-        run: echo "role=$aws_role  bucket=$s3_bucket"
+        run: echo "acct=${{ secrets.AWS_DEV_ACCT }} role=${{ secrets.AWS_ROLE }} bucket=$s3_bucket"
       - name: Configure AWS Credentials
         uses: aws-actions/configure-aws-credentials@v4
         with:
-          role-to-assume: $aws_role
+          role-to-assume: arn:aws:iam::${{ secrets.AWS_DEV_ACCT }}:role/${{ secrets.AWS_ROLE }}
           aws-region: us-east-2
       - name: Copy files to the test website with the AWS CLI
         run: |