Skip to content

Commit 4b3ec3f

Browse files
DimitarTachevDimitarTachev
committed
fix: add more signing validations
1 parent e65efeb commit 4b3ec3f

File tree

8 files changed

+83
-15
lines changed

8 files changed

+83
-15
lines changed

‎lib/commands/build.ts‎

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
import { ANDROID_RELEASE_BUILD_ERROR_MESSAGE, AndroidAppBundleMessages } from "../constants";
1+
import { ANDROID_RELEASE_BUILD_ERROR_MESSAGE, AndroidAppBundleMessages, ANDROID_APP_BUNDLE_SIGNING_ERROR_MESSAGE } from "../constants";
22
import { ValidatePlatformCommandBase } from "./command-base";
3+
import { hasValidAndroidSigning } from "../common/helpers";
34

45
export abstract class BuildCommandBase extends ValidatePlatformCommandBase {
56
constructor($options: IOptions,
@@ -123,8 +124,12 @@ export class BuildAndroidCommand extends BuildCommandBase implements ICommand {
123124
this.$androidBundleValidatorHelper.validateRuntimeVersion(this.$projectData);
124125
let canExecute = await super.canExecuteCommandBase(platform, { notConfiguredEnvOptions: { hideSyncToPreviewAppOption: true } });
125126
if (canExecute) {
126-
if (this.$options.release && (!this.$options.keyStorePath || !this.$options.keyStorePassword || !this.$options.keyStoreAlias || !this.$options.keyStoreAliasPassword)) {
127-
this.$errors.failWithHelp(ANDROID_RELEASE_BUILD_ERROR_MESSAGE);
127+
if ((this.$options.release || this.$options.aab) && !hasValidAndroidSigning(this.$options)) {
128+
if (this.$options.release) {
129+
this.$errors.failWithHelp(ANDROID_RELEASE_BUILD_ERROR_MESSAGE);
130+
} else {
131+
this.$errors.failWithHelp(ANDROID_APP_BUNDLE_SIGNING_ERROR_MESSAGE);
132+
}
128133
}
129134

130135
canExecute = await super.validateArgs(args, platform);

‎lib/commands/debug.ts‎

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
import { cache } from "../common/decorators";
22
import { ValidatePlatformCommandBase } from "./command-base";
3+
import { hasValidAndroidSigning } from "../common/helpers";
4+
import { ANDROID_RELEASE_BUILD_ERROR_MESSAGE, ANDROID_APP_BUNDLE_SIGNING_ERROR_MESSAGE } from "../constants";
35

46
export class DebugPlatformCommand extends ValidatePlatformCommandBase implements ICommand {
57
public allowedParameters: ICommandParameter[] = [];
@@ -155,7 +157,8 @@ export class DebugAndroidCommand implements ICommand {
155157
private $devicePlatformsConstants: Mobile.IDevicePlatformsConstants,
156158
private $injector: IInjector,
157159
private $projectData: IProjectData,
158-
private $markingModeService: IMarkingModeService) {
160+
private $markingModeService: IMarkingModeService,
161+
private $options: IOptions) {
159162
this.$projectData.initializeProjectData();
160163
}
161164

@@ -164,8 +167,18 @@ export class DebugAndroidCommand implements ICommand {
164167
return this.debugPlatformCommand.execute(args);
165168
}
166169
public async canExecute(args: string[]): Promise<boolean> {
167-
const result = await this.debugPlatformCommand.canExecute(args);
168-
return result;
170+
const canExecuteBase = await this.debugPlatformCommand.canExecute(args);
171+
if (canExecuteBase) {
172+
if (this.$options.aab && !hasValidAndroidSigning(this.$options)) {
173+
if (this.$options.release) {
174+
this.$errors.failWithHelp(ANDROID_RELEASE_BUILD_ERROR_MESSAGE);
175+
} else {
176+
this.$errors.failWithHelp(ANDROID_APP_BUNDLE_SIGNING_ERROR_MESSAGE);
177+
}
178+
}
179+
}
180+
181+
return canExecuteBase;
169182
}
170183

171184
public platform = this.$devicePlatformsConstants.Android;

‎lib/commands/deploy.ts‎

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
1-
import { ANDROID_RELEASE_BUILD_ERROR_MESSAGE } from "../constants";
1+
import { ANDROID_RELEASE_BUILD_ERROR_MESSAGE, ANDROID_APP_BUNDLE_SIGNING_ERROR_MESSAGE } from "../constants";
22
import { ValidatePlatformCommandBase } from "./command-base";
33
import { DeployCommandHelper } from "../helpers/deploy-command-helper";
4+
import { hasValidAndroidSigning } from "../common/helpers";
45

56
export class DeployOnDeviceCommand extends ValidatePlatformCommandBase implements ICommand {
67
public allowedParameters: ICommandParameter[] = [];
@@ -47,8 +48,12 @@ export class DeployOnDeviceCommand extends ValidatePlatformCommandBase implement
4748
return false;
4849
}
4950

50-
if (this.$mobileHelper.isAndroidPlatform(platform) && this.$options.release && (!this.$options.keyStorePath || !this.$options.keyStorePassword || !this.$options.keyStoreAlias || !this.$options.keyStoreAliasPassword)) {
51-
this.$errors.failWithHelp(ANDROID_RELEASE_BUILD_ERROR_MESSAGE);
51+
if (this.$mobileHelper.isAndroidPlatform(platform) && (this.$options.release || this.$options.aab) && !hasValidAndroidSigning(this.$options)) {
52+
if (this.$options.release) {
53+
this.$errors.failWithHelp(ANDROID_RELEASE_BUILD_ERROR_MESSAGE);
54+
} else {
55+
this.$errors.failWithHelp(ANDROID_APP_BUNDLE_SIGNING_ERROR_MESSAGE);
56+
}
5257
}
5358

5459
const result = await super.canExecuteCommandBase(platform, { validateOptions: true });

‎lib/commands/run.ts‎

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import { ERROR_NO_VALID_SUBCOMMAND_FORMAT } from "../common/constants";
2-
import { ANDROID_RELEASE_BUILD_ERROR_MESSAGE } from "../constants";
2+
import { ANDROID_RELEASE_BUILD_ERROR_MESSAGE, ANDROID_APP_BUNDLE_SIGNING_ERROR_MESSAGE } from "../constants";
33
import { cache } from "../common/decorators";
4+
import { hasValidAndroidSigning } from "../common/helpers";
45

56
export class RunCommandBase implements ICommand {
67
private liveSyncCommandHelperAdditionalOptions: ILiveSyncCommandHelperAdditionalOptions = <ILiveSyncCommandHelperAdditionalOptions>{};
@@ -126,8 +127,12 @@ export class RunAndroidCommand implements ICommand {
126127
this.$errors.fail(`Applications for platform ${this.$devicePlatformsConstants.Android} can not be built on this OS`);
127128
}
128129

129-
if (this.$options.release && (!this.$options.keyStorePath || !this.$options.keyStorePassword || !this.$options.keyStoreAlias || !this.$options.keyStoreAliasPassword)) {
130-
this.$errors.failWithHelp(ANDROID_RELEASE_BUILD_ERROR_MESSAGE);
130+
if ((this.$options.release || this.$options.aab) && !hasValidAndroidSigning(this.$options)) {
131+
if (this.$options.release) {
132+
this.$errors.failWithHelp(ANDROID_RELEASE_BUILD_ERROR_MESSAGE);
133+
} else {
134+
this.$errors.failWithHelp(ANDROID_APP_BUNDLE_SIGNING_ERROR_MESSAGE);
135+
}
131136
}
132137

133138
return this.$platformValidationService.validateOptions(this.$options.provision, this.$options.teamId, this.$projectData, this.$devicePlatformsConstants.Android.toLowerCase());

‎lib/commands/test.ts‎

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,6 @@
1+
import { hasValidAndroidSigning } from "../common/helpers";
2+
import { ANDROID_RELEASE_BUILD_ERROR_MESSAGE, ANDROID_APP_BUNDLE_SIGNING_ERROR_MESSAGE } from "../constants";
3+
14
abstract class TestCommandBase {
25
public allowedParameters: ICommandParameter[] = [];
36
public dashedOptions = {
@@ -111,6 +114,21 @@ class TestAndroidCommand extends TestCommandBase implements ICommand {
111114
await this.$markingModeService.handleMarkingModeFullDeprecation({ projectDir: this.$projectData.projectDir, skipWarnings: true });
112115
await super.execute(args);
113116
}
117+
118+
async canExecute(args: string[]): Promise<boolean> {
119+
const canExecuteBase = await super.canExecute(args);
120+
if (canExecuteBase) {
121+
if ((this.$options.release || this.$options.aab) && !hasValidAndroidSigning(this.$options)) {
122+
if (this.$options.release) {
123+
this.$errors.failWithHelp(ANDROID_RELEASE_BUILD_ERROR_MESSAGE);
124+
} else {
125+
this.$errors.failWithHelp(ANDROID_APP_BUNDLE_SIGNING_ERROR_MESSAGE);
126+
}
127+
}
128+
}
129+
130+
return canExecuteBase;
131+
}
114132
}
115133

116134
class TestIosCommand extends TestCommandBase implements ICommand {

‎lib/common/helpers.ts‎

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -736,4 +736,20 @@ export function annotate(fn: any) {
736736
return $inject;
737737
}
738738

739+
740+
/**
741+
* Returns true if all Android signing options are provided, false otherwise.
742+
* @param {IAndroidSigningData} signingData The signing data to be validated.
743+
* @return {void}
744+
*/
745+
export function hasValidAndroidSigning(signingData: Partial<IAndroidSigningData>): boolean {
746+
const isValid = signingData &&
747+
signingData.keyStorePath &&
748+
signingData.keyStorePassword &&
749+
signingData.keyStoreAlias &&
750+
signingData.keyStoreAliasPassword;
751+
752+
return !!isValid;
753+
}
754+
739755
//--- end part copied from AngularJS

‎lib/constants.ts‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,9 @@ export const DEBUGGER_DETACHED_EVENT_NAME = "debuggerDetached";
150150
export const VERSION_STRING = "version";
151151
export const INSPECTOR_CACHE_DIRNAME = "ios-inspector";
152152
export const POST_INSTALL_COMMAND_NAME = "post-install-cli";
153-
export const ANDROID_RELEASE_BUILD_ERROR_MESSAGE = "When producing a release build, you need to specify all --key-store-* options.";
153+
const ANDROID_SIGNING_REQUIRED_MESSAGE = "you need to specify all --key-store-* options.";
154+
export const ANDROID_RELEASE_BUILD_ERROR_MESSAGE = `When producing a release build, ${ANDROID_SIGNING_REQUIRED_MESSAGE}`;
155+
export const ANDROID_APP_BUNDLE_SIGNING_ERROR_MESSAGE = `When producing Android App Bundle, ${ANDROID_SIGNING_REQUIRED_MESSAGE}`;
154156
export const CACACHE_DIRECTORY_NAME = "_cacache";
155157

156158
export const FILES_CHANGE_EVENT_NAME = "filesChangeEvent";

‎lib/services/android/android-bundle-tool-service.ts‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,12 @@ export class AndroidBundleToolService implements IAndroidBundleToolService {
1212
}
1313

1414
public async buildApks(options: IBuildApksOptions): Promise<void> {
15-
if (!options.signingData) {
16-
this.$errors.fail(`Unable to build "apks" without a signing information.`);
15+
if (!options.signingData ||
16+
!options.signingData.keyStorePath ||
17+
!options.signingData.keyStorePassword ||
18+
!options.signingData.keyStoreAlias ||
19+
!options.signingData.keyStoreAliasPassword) {
20+
this.$errors.fail(`Unable to build "apks" without a full signing information.`);
1721
}
1822

1923
const aabToolResult = await this.execBundleTool([

0 commit comments

Comments
 (0)