fix: sanitize environment variables for PR SHA and number (#206)

Author: AdnaneKhan

.github/workflows/externalPR.yml

@@ -38,8 +38,8 @@ jobs:
       - run: unzip pr.zip
       - name: outputs
         run: |-
-          echo "PR=$(cat NR)" >> $GITHUB_ENV
-          echo "SHA=$(cat SHA)" >> $GITHUB_ENV
+          echo "PR=$(cat NR | tr -cd '0-9')" >> $GITHUB_ENV
+          echo "SHA=$(cat SHA | tr -cd 'a-fA-F0-9')" >> $GITHUB_ENV
       - name: cobertura-report-unit-test
         uses: 5monkeys/cobertura-action@v14
         continue-on-error: true

.github/workflows/maven.yml

@@ -24,7 +24,7 @@ jobs:
       - name: Build
         run: mvn -U -ntp clean verify
       - name: Upload Coverage
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: Coverage Report
           path: target/jacoco-report
@@ -39,7 +39,7 @@ jobs:
           cp target/jacoco-report/cobertura.xml ./pr/jacoco-report/cobertura.xml
         if: github.event_name == 'pull_request'
       - name: Upload files
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: pr
           path: pr/