Merge pull request #34 from actions/change-file-permissions

Before tarring site, set file read permissions
Fix the mac shell script
2022-11-15 12:37:36 -05:00 · 2022-11-15 10:18:49 -05:00 · 2022-11-14 11:41:47 -05:00 · 2022-11-08 10:53:50 -05:00 · 2022-11-07 13:02:01 -05:00 · 2022-11-01 16:53:27 -07:00
2 changed files with 11 additions and 3 deletions
@@ -8,7 +8,7 @@ jobs:
  draft-release:
    runs-on: ubuntu-latest
    steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
      - uses: release-drafter/release-drafter@v5
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -14,9 +14,13 @@ runs:
  using: composite
  steps:
    - name: Archive artifact
-      shell: bash
+      shell: sh
      if: runner.os == 'Linux'
      run: |
+        for f in $(chmod -c -R +r . | awk '{print substr($3, 2, length($3)-2)}')
+        do
+          echo "::warning::Added read permission to $f"
+        done
        tar \
          --dereference --hard-dereference \
          --directory "$INPUT_PATH" \
@@ -29,9 +33,13 @@ runs:

    # Switch to gtar (GNU tar instead of bsdtar which is the default in the MacOS runners so we can use --hard-dereference)
    - name: Archive artifact
-      shell: bash
+      shell: sh
      if: runner.os == 'macOS'
      run: |
+        for f in $(chmod -v -R +r .)
+        do
+          echo "::warning::Added read permission to $f"
+        done
        gtar \
          --dereference --hard-dereference \
          --directory "$INPUT_PATH" \
Author	SHA1	Message	Date
Jess Bees	4506126207	Merge pull request #34 from actions/change-file-permissions Before tarring site, set file read permissions	2022-11-15 12:37:36 -05:00
Jess Bees	6de1c7ae49	Fix the mac shell script	2022-11-15 10:18:49 -05:00
Jess Bees	de1b81a30a	Use macos's non-gnu version of chmod	2022-11-14 11:41:47 -05:00
Jess Bees	61a978d261	Send a workflow warning when changing file permissions	2022-11-08 10:53:50 -05:00
Jess Bees	3cb2c0f93b	Before tarring site, set file read permissions Pages DFS needs the "group read" permission set on any file it serves, so add read permission before tarring to prevent unreadable files from being deployed. This only applies to files that are built on an actions runner, not files that are cloned from a git repoitory, because git's permissions are limited to the equivalents of chmod 644 and 755.	2022-11-07 13:02:01 -05:00
Yoann Chaudet	e02d70a4e3	Merge pull request #32 from actions/bump-actions Update actions/checkout	2022-11-01 16:53:27 -07:00
Yoann Chaudet	73b4a0753a	Merge branch 'main' into bump-actions	2022-11-01 16:49:06 -07:00
Yoann Chaudet	aff0f4398b	Merge pull request #14 from ncfavier/patch-1 Fall back to `sh` on non-Windows	2022-11-01 16:30:32 -07:00
Yoann Chaudet	42a1761ba2	Update actions/checkout	2022-10-31 16:25:59 -07:00
Naïm Favier	12e73a1efa	Fall back to `sh` on non-Windows Fixes https://github.com/actions/upload-pages-artifact/issues/13	2022-10-29 11:59:56 +02:00