Skip to content

Commit

Permalink
FIX Remove broken code to copy diff files to artifacts
Browse files Browse the repository at this point in the history
  • Loading branch information
emteknetnz committed May 9, 2024
1 parent 2dc432f commit b3d4b59
Showing 1 changed file with 3 additions and 13 deletions.
16 changes: 3 additions & 13 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -153,6 +153,9 @@ runs:
- name: Run JS tests
if: ${{ inputs.js == 'true' }}
shell: bash
env:
GITHUB_REF_NAME: ${{ github.ref_name }}
GITHUB_REPOSITORY: ${{ github.repository }}
run: |
echo "Running JS tests"
if [[ ! -f package.json ]]; then
Expand Down Expand Up @@ -216,24 +219,11 @@ runs:
echo "git diff found modified files when it should not have:"
echo $GIT_DIFF
echo "sha1sum of files that are different:"
COPIED_COUNT=0
for FILEPATH in $(git diff --cached --name-only); do
if [[ -f $FILEPATH ]]; then
sha1sum $FILEPATH
# Only copy if the file is less than 10 megabytes to prevent malicous behaviour
# Only copy a max of 10 files, also to prevent malicious behaviour
MEGABYTES=$(ls -l --b=M "$DIST_DIR/$FILEPATH" | cut -d " " -f5 | sed "s/M//")
if (($MEGABYTES >= 10)); then
echo "File $FILEPATH is larger than 10 megabytes, not copying"
elif (($COPIED_COUNT >= 10)); then
echo "More than 10 files have been copied, not copying $FILEPATH"
else
cp "$DIST_DIR/$FILEPATH" artifacts
COPIED_COUNT=$((COPIED_COUNT+1))
fi
fi
done
echo "Files that are different have been copied to artifacts directory"
exit 1
fi
fi
Expand Down

0 comments on commit b3d4b59

Please sign in to comment.