Merge pull request #53 from lriccardo/master

Add internal sharing download url to outputs and env

README.md

@@ -18,9 +18,14 @@ The service account json in plain text, provided via a secret, etc.
 
 **Required:** The package name, or Application Id, of the app you are uploading
 
-### `releaseFiles`
+### `releaseFile`
+**DEPRECATED:** Please switch to using `releaseFiles` as this will be removed in the future  
+The Android release file to upload (.apk or .aab) 
 
-**Required:** The Android release file(s) to upload (.apk or .aab). Multiple files are separated by ','.
+### `releaseFiles`
+**CAVEAT:** Either this or `releaseFile` are required  
+
+The Android release file(s) to upload (.apk or .aab). Multiple files are separated by ','.
 
 ### `releaseName`
 
@@ -29,14 +34,14 @@ The release name. Not required to be unique. If not set, the name is generated f
 ### `track`
 
 **Required:** The track in which you want to assign the uploaded app.  
-**Default:** `production`  
+**Default:** `production`   
 _Values:_ `alpha`, `beta`, `internal`, `production`, `internalsharing`
 
 ### `inAppUpdatePriority`
 
 In-app update priority of the release. All newly added APKs in the release will be considered at this priority. Can take values in the range [0, 5], with 5 the highest priority.
 
-**Default:** `0`
+**Default:** `0`  
 _Values:_ `[0, 5]`
 
 ### `userFraction`
@@ -67,9 +72,21 @@ The mapping.txt file used to de-obfuscate your stack traces from crash reports
 
 ## Outputs
 
+### `releaseName`
+
+The name of the release published to Google Play, which is generated on commit by default
+
+### `internalSharingDownloadUrls`
+
+A JSON list containing the download urls for every release file uploaded using the `track` `internalsharing`
+
+### ENV VAR: `INTERNAL_SHARING_DOWNLOAD_URLS`
+
+The environment variable that is set when using the `track` `internalsharing`
+
 ### `internalSharingDownloadUrl`
 
-The download url for an app that was uploaded using the `track` `internalsharing`
+The download url for the last release file uploaded using the `track` `internalsharing`, useful when a single release file is uploaded
 
 ### ENV VAR: `INTERNAL_SHARING_DOWNLOAD_URL`
 

action.yml

@@ -1,41 +1,44 @@
 name: 'Upload Android Release to Play Store'
-description: 'An action to upload a signed Android release (.apk or .aab) to the Google Play Store Developer Console'
+description: "An action to upload a signed Android release (.apk or .aab) to the Google Play Store Developer Console"
 author: 'r0adkll'
 branding:
   icon: 'truck'
   color: 'orange'
 inputs:
   serviceAccountJson:
-    description: 'The service account json private key file to authorize the upload request'
+    description: "The service account json private key file to authorize the upload request"
     required: false
   serviceAccountJsonPlainText:
-    description: 'The raw json text of the service account private key to authorize the upload request'
+    description: "The raw json text of the service account private key to authorize the upload request"
     required: false
   packageName:
-    description: 'The package name, or Application Id, of the app you are uploading'
+    description: "The package name, or Application Id, of the app you are uploading"
     required: true
+  releaseFile:
+    description: "The Android release file to upload (.apk or .aab)"
+    required: false
   releaseFiles:
     description: "The Android release file(s) to upload (.apk or .aab). Separated by  a ',' for multiple artifacts"
-    required: true
+    required: false
   releaseName:
-    description: 'The name of this release. If not set it's generated automatically from the APKs versionName'
+    description: "The name of this release. If not set it's generated automatically from the APKs versionName"
     required: false
   track:
-    description: 'The track in which you want to assign the uploaded app.'
+    description: "The track in which you want to assign the uploaded app."
     default: 'production'
     required: true
   inAppUpdatePriority:
-    description: 'In-app update priority of the release. All newly added APKs in the release will be considered at this priority. Can take values in the range [0, 5], with 5 the highest priority. Defaults to 0.'
+    description: "In-app update priority of the release. All newly added APKs in the release will be considered at this priority. Can take values in the range [0, 5], with 5 the highest priority. Defaults to 0."
     default: '0'
     required: false
   userFraction:
-    description: 'Portion of users who should get the staged version of the app. Accepts values between 0.0 and 1.0 (exclusive-exclusive).'
+    description: "Portion of users who should get the staged version of the app. Accepts values between 0.0 and 1.0 (exclusive-exclusive)."
     required: false
   whatsNewDirectory:
-    description: 'The directory of localized whats new files'
+    description: "The directory of localized whats new files"
     required: false
   mappingFile:
-    description: 'The mapping.txt file used to de-obfuscate your stack traces from crash reports'
+    description: "The mapping.txt file used to de-obfuscate your stack traces from crash reports"
     required: false
 outputs:
   internalSharingDownloadUrl:

lib/edits.js

@@ -14,7 +14,7 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 var __importStar = (this && this.__importStar) || function (mod) {
     if (mod && mod.__esModule) return mod;
     var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
     __setModuleDefault(result, mod);
     return result;
 };
@@ -40,38 +40,61 @@ function uploadToPlayStore(options, releaseFiles) {
         // Check the 'track' for 'internalsharing', if so switch to a non-track api
         if (options.track === 'internalsharing') {
             core.debug("Track is Internal app sharing, switch to special upload api");
+            let downloadUrls = []
             for (const releaseFile of releaseFiles) {
                 core.debug(`Uploading ${releaseFile}`);
-                yield uploadInternalSharingRelease(options, releaseFile).catch(reason => {
-                    core.setFailed(reason);
-                    return Promise.reject(reason);
-                });
+                yield uploadInternalSharingRelease(options, releaseFile)
+                    .then(downloadUrl => {
+                        downloadUrls.push(downloadUrl)
+                    })
+                    .catch(reason => {
+                        core.setFailed(reason);
+                        return Promise.reject(reason);
+                    });
             }
+            core.setOutput("internalSharingDownloadUrls", downloadUrls);
+            core.exportVariable("INTERNAL_SHARING_DOWNLOAD_URLS", downloadUrls);
         }
         else {
             // Create a new Edit
+            core.info(`Creating a new Edit for this release`);
             const appEdit = yield androidPublisher.edits.insert({
                 auth: options.auth,
                 packageName: options.applicationId
             });
             // Validate the given track
+            core.info(`Validating track '${options.track}'`);
             yield validateSelectedTrack(appEdit.data, options).catch(reason => {
                 core.setFailed(reason);
                 return Promise.reject(reason);
             });
             // Upload artifacts to Google Play, and store their version codes
-            var versionCodes = new Array();
+            const versionCodes = new Array();
             for (const releaseFile of releaseFiles) {
-                core.debug(`Uploading ${releaseFile}`);
+                core.info(`Uploading ${releaseFile}`);
                 const versionCode = yield uploadRelease(appEdit.data, options, releaseFile).catch(reason => {
                     core.setFailed(reason);
                     return Promise.reject(reason);
                 });
                 versionCodes.push(versionCode);
             }
+            core.info(`Successfully uploaded ${versionCodes.length} artifacts`);
             // Add the uploaded artifacts to the Edit track
+            core.info(`Adding ${versionCodes.length} artifacts to release on '${options.track}' track`);
             const track = yield addReleasesToTrack(appEdit.data, options, versionCodes);
+            core.debug(`Track: ${track}`);
+            // Extract the release name for the response
+            let trackReleaseName;
+            if (track.releases && track.releases.length > 0) {
+                const trackRelease = track.releases[0];
+                trackReleaseName = trackRelease.name;
+                core.debug(`Pulled track release name from update: ${trackReleaseName}`);
+            }
+            else {
+                core.debug(`Didn't find any releases on the track`);
+            }
             // Commit the pending Edit
+            core.info(`Committing the Edit`);
             const res = yield androidPublisher.edits.commit({
                 auth: options.auth,
                 editId: appEdit.data.id,
@@ -80,6 +103,14 @@ function uploadToPlayStore(options, releaseFiles) {
             // Simple check to see whether commit was successful
             if (res.data.id != null) {
                 core.debug(`Successfully committed ${res.data.id}`);
+                const name = options.name || trackReleaseName;
+                core.info(`Successfully committed release ${name}`);
+                if (name) {
+                    core.setOutput("releaseName", name);
+                }
+                else {
+                    core.debug("Unable to set 'releaseName' output.");
+                }
                 return Promise.resolve(res.data.id);
             }
             else {
@@ -94,11 +125,15 @@ function uploadInternalSharingRelease(options, releaseFile) {
     return __awaiter(this, void 0, void 0, function* () {
         if (releaseFile.endsWith('.apk')) {
             const res = yield internalSharingUploadApk(options, releaseFile);
+            core.setOutput("internalSharingDownloadUrl", res.downloadUrl);
+            core.exportVariable("INTERNAL_SHARING_DOWNLOAD_URL", res.downloadUrl);
             console.log(`${releaseFile} uploaded to Internal Sharing, download it with ${res.downloadUrl}`);
             return Promise.resolve(res.downloadUrl);
         }
         else if (releaseFile.endsWith('.aab')) {
             const res = yield internalSharingUploadBundle(options, releaseFile);
+            core.setOutput("internalSharingDownloadUrl", res.downloadUrl);
+            core.exportVariable("INTERNAL_SHARING_DOWNLOAD_URL", res.downloadUrl);
             console.log(`${releaseFile} uploaded to Internal Sharing, download it with ${res.downloadUrl}`);
             return Promise.resolve(res.downloadUrl);
         }

lib/main.js

@@ -14,7 +14,7 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 var __importStar = (this && this.__importStar) || function (mod) {
     if (mod && mod.__esModule) return mod;
     var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
     __setModuleDefault(result, mod);
     return result;
 };
@@ -36,12 +36,14 @@ const auth = new google.auth.GoogleAuth({
     scopes: ['https://www.googleapis.com/auth/androidpublisher']
 });
 function run() {
+    var _a, _b;
     return __awaiter(this, void 0, void 0, function* () {
         try {
             const serviceAccountJson = core.getInput('serviceAccountJson', { required: false });
             const serviceAccountJsonRaw = core.getInput('serviceAccountJsonPlainText', { required: false });
             const packageName = core.getInput('packageName', { required: true });
-            const releaseFiles = core.getInput('releaseFiles', { required: true }).split(',').filter(x => x !== '');
+            const releaseFile = core.getInput('releaseFile', { required: false });
+            const releaseFiles = ((_b = (_a = core.getInput('releaseFiles', { required: false })) === null || _a === void 0 ? void 0 : _a.split(',')) === null || _b === void 0 ? void 0 : _b.filter(x => x !== '')) || [];
             const releaseName = core.getInput('releaseName', { required: false });
             const track = core.getInput('track', { required: true });
             const inAppUpdatePriority = core.getInput('inAppUpdatePriority', { required: false });
@@ -52,6 +54,7 @@ function run() {
             if (!serviceAccountJson && !serviceAccountJsonRaw) {
                 console.log("No service account json key provided!");
                 core.setFailed("You must provide one of 'serviceAccountJson' or 'serviceAccountJsonPlainText' to use this action");
+                return;
             }
             // If the user has provided the raw plain text via secrets, or w/e, then write to file and
             // set appropriate env variable for the auth
@@ -80,7 +83,7 @@ function run() {
             }
             // Validate the inAppUpdatePriority to be a valid number in within [0, 5]
             let inAppUpdatePriorityInt = parseInt(inAppUpdatePriority);
-            if (~isNaN(inAppUpdatePriorityInt)) {
+            if (!isNaN(inAppUpdatePriorityInt)) {
                 if (inAppUpdatePriorityInt < 0 || inAppUpdatePriorityInt > 5) {
                     core.setFailed('inAppUpdatePriority must be between 0 and 5, inclusive-inclusive');
                     return;
@@ -89,13 +92,32 @@ function run() {
             else {
                 inAppUpdatePriorityInt = undefined;
             }
-            // Check release files
-            for (const releaseFile of releaseFiles) {
+            // Check release files while maintaining backward compatibility
+            let validatedReleaseFiles = [];
+            if (releaseFiles.length == 0 && !releaseFile) {
+                core.setFailed(`You must provide either 'releaseFile' or 'releaseFiles' in your configuration.`);
+                return;
+            }
+            else if (releaseFiles.length == 0 && releaseFile) {
+                core.warning(`WARNING!! 'releaseFile' is deprecated and will be removed in a future release. Please migrate to 'releaseFiles'.`);
                 core.debug(`Validating ${releaseFile} exists`);
                 if (!fs.existsSync(releaseFile)) {
                     core.setFailed(`Unable to find release file @ ${releaseFile}`);
                     return;
                 }
+                else {
+                    validatedReleaseFiles = [releaseFile];
+                }
+            }
+            else if (releaseFiles.length > 0) {
+                for (const file of releaseFiles) {
+                    core.debug(`Validating ${file} exists`);
+                    if (!fs.existsSync(file)) {
+                        core.setFailed(`Unable to find release file @ ${file}`);
+                        return;
+                    }
+                }
+                validatedReleaseFiles = releaseFiles;
             }
             if (whatsNewDir != undefined && whatsNewDir.length > 0 && !fs.existsSync(whatsNewDir)) {
                 core.setFailed(`Unable to find 'whatsnew' directory @ ${whatsNewDir}`);
@@ -106,7 +128,7 @@ function run() {
                 return;
             }
             const authClient = yield auth.getClient();
-            yield edits_1.uploadToPlayStore({
+            const result = yield edits_1.uploadToPlayStore({
                 auth: authClient,
                 applicationId: packageName,
                 track: track,
@@ -115,8 +137,8 @@ function run() {
                 whatsNewDir: whatsNewDir,
                 mappingFile: mappingFile,
                 name: releaseName
-            }, releaseFiles);
-            console.log(`Finished uploading to the Play Store`);
+            }, validatedReleaseFiles);
+            console.log(`Finished uploading to the Play Store: ${result}`);
         }
         catch (error) {
             core.setFailed(error.message);

lib/whatsnew.js

@@ -14,7 +14,7 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 var __importStar = (this && this.__importStar) || function (mod) {
     if (mod && mod.__esModule) return mod;
     var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
     __setModuleDefault(result, mod);
     return result;
 };

package.json

@@ -1,6 +1,6 @@
 {
   "name": "upload-google-play",
-  "version": "1.0.6",
+  "version": "1.0.10",
   "private": true,
   "description": "Upload an Android release (.apk or .aab) to Google Play Console",
   "main": "lib/main.js",

src/edits.ts

@@ -33,41 +33,64 @@ export async function uploadToPlayStore(options: EditOptions, releaseFiles: stri
     // Check the 'track' for 'internalsharing', if so switch to a non-track api
     if (options.track === 'internalsharing') {
         core.debug("Track is Internal app sharing, switch to special upload api")
+        let downloadUrls: string[] = []
         for (const releaseFile of releaseFiles) {
             core.debug(`Uploading ${releaseFile}`);
-            await uploadInternalSharingRelease(options, releaseFile).catch(reason => {
-                core.setFailed(reason);
-                return Promise.reject(reason);
-            });
+            await uploadInternalSharingRelease(options, releaseFile)
+                .then(downloadUrl => {
+                    downloadUrls.push(downloadUrl as string)
+                })
+                .catch(reason => {
+                    core.setFailed(reason);
+                    return Promise.reject(reason);
+                });
         }
+        core.setOutput("internalSharingDownloadUrls", downloadUrls);
+        core.exportVariable("INTERNAL_SHARING_DOWNLOAD_URLS", downloadUrls);
     } else {
         // Create a new Edit
+        core.info(`Creating a new Edit for this release`)
         const appEdit = await androidPublisher.edits.insert({
             auth: options.auth,
             packageName: options.applicationId
         });
 
         // Validate the given track
+        core.info(`Validating track '${options.track}'`)
         await validateSelectedTrack(appEdit.data, options).catch(reason => {
             core.setFailed(reason);
             return Promise.reject(reason);
         });
 
         // Upload artifacts to Google Play, and store their version codes
-        var versionCodes = new Array<number>();
+        const versionCodes = new Array<number>();
         for (const releaseFile of releaseFiles) {
-            core.debug(`Uploading ${releaseFile}`);
+            core.info(`Uploading ${releaseFile}`);
             const versionCode = await uploadRelease(appEdit.data, options, releaseFile).catch(reason => {
                 core.setFailed(reason);
                 return Promise.reject(reason);
             });
             versionCodes.push(versionCode!);
         }
+        core.info(`Successfully uploaded ${versionCodes.length} artifacts`)
 
         // Add the uploaded artifacts to the Edit track
+        core.info(`Adding ${versionCodes.length} artifacts to release on '${options.track}' track`)
         const track = await addReleasesToTrack(appEdit.data, options, versionCodes);
+        core.debug(`Track: ${track}`);
+
+        // Extract the release name for the response
+        let trackReleaseName: string | undefined | null;
+        if (track.releases && track.releases.length > 0) {
+            const trackRelease = track.releases[0];
+            trackReleaseName = trackRelease.name;
+            core.debug(`Pulled track release name from update: ${trackReleaseName}`)
+        } else {
+            core.debug(`Didn't find any releases on the track`)
+        }
 
         // Commit the pending Edit
+        core.info(`Committing the Edit`)
         const res = await androidPublisher.edits.commit({
             auth: options.auth,
             editId: appEdit.data.id!,
@@ -77,6 +100,13 @@ export async function uploadToPlayStore(options: EditOptions, releaseFiles: stri
         // Simple check to see whether commit was successful
         if (res.data.id != null) {
             core.debug(`Successfully committed ${res.data.id}`);
+            const name = options.name || trackReleaseName;
+            core.info(`Successfully committed release ${name}`)
+            if (name) {
+                core.setOutput("releaseName", name);
+            } else {
+                core.debug("Unable to set 'releaseName' output.");
+            }
             return Promise.resolve(res.data.id!);
         } else {
             core.setFailed(`Error ${res.status}: ${res.statusText}`);
@@ -88,10 +118,14 @@ export async function uploadToPlayStore(options: EditOptions, releaseFiles: stri
 async function uploadInternalSharingRelease(options: EditOptions, releaseFile: string): Promise<string | undefined | null> {
     if (releaseFile.endsWith('.apk')) {
         const res = await internalSharingUploadApk(options, releaseFile)
+        core.setOutput("internalSharingDownloadUrl", res.downloadUrl);
+        core.exportVariable("INTERNAL_SHARING_DOWNLOAD_URL", res.downloadUrl);
         console.log(`${releaseFile} uploaded to Internal Sharing, download it with ${res.downloadUrl}`)
         return Promise.resolve(res.downloadUrl)
     } else if (releaseFile.endsWith('.aab')) {
         const res = await internalSharingUploadBundle(options, releaseFile)
+        core.setOutput("internalSharingDownloadUrl", res.downloadUrl);
+        core.exportVariable("INTERNAL_SHARING_DOWNLOAD_URL", res.downloadUrl);
         console.log(`${releaseFile} uploaded to Internal Sharing, download it with ${res.downloadUrl}`)
         return Promise.resolve(res.downloadUrl)
     } else {
@@ -237,4 +271,4 @@ async function uploadBundle(appEdit: AppEdit, options: EditOptions, bundleReleas
     });
 
     return res.data
-}
+}

src/main.ts

@@ -12,7 +12,10 @@ async function run() {
         const serviceAccountJson = core.getInput('serviceAccountJson', { required: false });
         const serviceAccountJsonRaw = core.getInput('serviceAccountJsonPlainText', { required: false});
         const packageName = core.getInput('packageName', { required: true });
-        const releaseFiles = core.getInput('releaseFiles', { required: true }).split(',').filter(x => x !== '');
+        const releaseFile = core.getInput('releaseFile', { required: false });
+        const releaseFiles = core.getInput('releaseFiles', { required: false })
+            ?.split(',')
+            ?.filter(x => x !== '') || [];
         const releaseName = core.getInput('releaseName', { required: false });
         const track = core.getInput('track', { required: true });
         const inAppUpdatePriority = core.getInput('inAppUpdatePriority', { required: false });
@@ -24,6 +27,7 @@ async function run() {
         if (!serviceAccountJson && !serviceAccountJsonRaw) {
             console.log("No service account json key provided!");
             core.setFailed("You must provide one of 'serviceAccountJson' or 'serviceAccountJsonPlainText' to use this action");
+            return;
         }
 
         // If the user has provided the raw plain text via secrets, or w/e, then write to file and
@@ -56,7 +60,7 @@ async function run() {
 
         // Validate the inAppUpdatePriority to be a valid number in within [0, 5]
         let inAppUpdatePriorityInt: number | undefined = parseInt(inAppUpdatePriority);
-        if (~isNaN(inAppUpdatePriorityInt)) {
+        if (!isNaN(inAppUpdatePriorityInt)) {
             if (inAppUpdatePriorityInt < 0 || inAppUpdatePriorityInt > 5) {
                 core.setFailed('inAppUpdatePriority must be between 0 and 5, inclusive-inclusive');
                 return;
@@ -65,13 +69,29 @@ async function run() {
             inAppUpdatePriorityInt = undefined;
         }
 
-        // Check release files
-        for (const releaseFile of releaseFiles) {
+        // Check release files while maintaining backward compatibility
+        let validatedReleaseFiles: string[] = [];
+        if (releaseFiles.length == 0 && !releaseFile) {
+            core.setFailed(`You must provide either 'releaseFile' or 'releaseFiles' in your configuration.`);
+            return;
+        } else if (releaseFiles.length == 0 && releaseFile) {
+            core.warning(`WARNING!! 'releaseFile' is deprecated and will be removed in a future release. Please migrate to 'releaseFiles'.`);
             core.debug(`Validating ${releaseFile} exists`)
             if (!fs.existsSync(releaseFile)) {
                 core.setFailed(`Unable to find release file @ ${releaseFile}`);
-                return
+                return;
+            } else {
+                validatedReleaseFiles = [releaseFile];
             }
+        } else if (releaseFiles.length > 0) {
+            for (const file of releaseFiles) {
+                core.debug(`Validating ${file} exists`)
+                if (!fs.existsSync(file)) {
+                    core.setFailed(`Unable to find release file @ ${file}`);
+                    return;
+                }
+            }
+            validatedReleaseFiles = releaseFiles;
         }
 
         if (whatsNewDir != undefined && whatsNewDir.length > 0 && !fs.existsSync(whatsNewDir)) {
@@ -86,7 +106,7 @@ async function run() {
 
         const authClient = await auth.getClient();
 
-        await uploadToPlayStore({
+        const result = await uploadToPlayStore({
             auth: authClient,
             applicationId: packageName,
             track: track,
@@ -95,9 +115,9 @@ async function run() {
             whatsNewDir: whatsNewDir,
             mappingFile: mappingFile,
             name: releaseName
-        }, releaseFiles);
+        }, validatedReleaseFiles);
 
-        console.log(`Finished uploading to the Play Store`)
+        console.log(`Finished uploading to the Play Store: ${result}`)
     } catch (error) {
         core.setFailed(error.message)
     }