Chown files to a custom user after building (fix file ownership) (#250)

* Chown files to a custom user after building * Better describe `chownFilesTo` input As suggested by @webbertakken: #250 (comment) Co-authored-by: Webber Takken <[email protected]> * Simplify chown step in `build.sh` Co-authored-by: Webber Takken <[email protected]> Co-authored-by: Webber Takken <[email protected]>
game-ci · May 1, 2021 · e31af20 · e31af20
1 parent a1f68ab
commit e31af20
Show file tree

Hide file tree

Showing 10 changed files with 532 additions and 493 deletions.
diff --git a/action.yml b/action.yml
@@ -106,6 +106,10 @@ inputs:
 
       Parameters must start with a hyphen (-) and may be followed by a value (without hyphen).
       Parameters without a value will be considered booleans (with a value of true).
+  chownFilesTo:
+    required: false
+    default: ''
+    description: 'User and optionally group (user or user:group or uid:gid) to give ownership of the resulting build artifacts'
   allowDirtyBuild:
     required: false
     default: ''

diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/dist/steps/build.sh b/dist/steps/build.sh
@@ -143,6 +143,12 @@ if [[ "$BUILD_TARGET" == "StandaloneOSX" ]]; then
   chmod +x $ADD_PERMISSIONS_PATH
 fi
 
+
+if [[ -n "$CHOWN_FILES_TO" ]]; then
+  chown -R $CHOWN_FILES_TO $BUILD_PATH_FULL
+  chown -R $CHOWN_FILES_TO $UNITY_PROJECT_PATH
+fi
+
 #
 # Results
 #

diff --git a/src/model/__mocks__/input.ts b/src/model/__mocks__/input.ts
@@ -10,6 +10,7 @@ export const mockGetFromUser = jest.fn().mockResolvedValue({
   buildMethod: undefined,
   buildVersion: '1.3.37',
   customParameters: '',
+  chownFilesTo: '',
 });
 
 export default {

diff --git a/src/model/aws.ts b/src/model/aws.ts
@@ -149,6 +149,10 @@ class AWS {
             name: 'CUSTOM_PARAMETERS',
             value: buildParameters.customParameters,
           },
+          {
+            name: 'CHOWN_FILES_TO',
+            value: buildParameters.chownFilesTo,
+          },
           {
             name: 'BUILD_TARGET',
             value: buildParameters.platform,

diff --git a/src/model/build-parameters.ts b/src/model/build-parameters.ts
@@ -33,6 +33,7 @@ class BuildParameters {
       androidKeyaliasName: Input.androidKeyaliasName,
       androidKeyaliasPass: Input.androidKeyaliasPass,
       customParameters: Input.customParameters,
+      chownFilesTo: Input.chownFilesTo,
       remoteBuildCluster: Input.remoteBuildCluster,
       awsStackName: Input.awsStackName,
       kubeConfig: Input.kubeConfig,

diff --git a/src/model/docker.ts b/src/model/docker.ts
@@ -36,6 +36,7 @@ class Docker {
       androidKeyaliasName,
       androidKeyaliasPass,
       customParameters,
+      chownFilesTo,
     } = parameters;
 
     const command = `docker run \
@@ -62,6 +63,7 @@ class Docker {
         --env ANDROID_KEYALIAS_NAME="${androidKeyaliasName}" \
         --env ANDROID_KEYALIAS_PASS="${androidKeyaliasPass}" \
         --env CUSTOM_PARAMETERS="${customParameters}" \
+        --env CHOWN_FILES_TO="${chownFilesTo}" \
         --env GITHUB_REF \
         --env GITHUB_SHA \
         --env GITHUB_REPOSITORY \

diff --git a/src/model/input.ts b/src/model/input.ts
@@ -85,6 +85,10 @@ class Input {
     return core.getInput('customParameters') || '';
   }
 
+  static get chownFilesTo() {
+    return core.getInput('chownFilesTo') || '';
+  }
+
   static get remoteBuildCluster() {
     return core.getInput('remoteBuildCluster') || '';
   }

diff --git a/src/model/kubernetes.ts b/src/model/kubernetes.ts
@@ -222,6 +222,10 @@ class Kubernetes {
                     name: 'CUSTOM_PARAMETERS',
                     value: this.buildParameters.customParameters,
                   },
+                  {
+                    name: 'CHOWN_FILES_TO',
+                    value: this.buildParameters.chownFilesTo,
+                  },
                   {
                     name: 'BUILD_TARGET',
                     value: this.buildParameters.platform,