Skip to content

Commit 30c82f0

Browse files
committed
Fail on invalid boolean for Develocity inputs
1 parent e3bc05f commit 30c82f0

File tree

1 file changed

+8
-12
lines changed

1 file changed

+8
-12
lines changed

sources/src/configuration.ts

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -217,23 +217,23 @@ export class BuildScanConfig {
217217
}
218218

219219
getDevelocityInjectionEnabled(): boolean | undefined {
220-
return getMaybeBooleanInput('develocity-injection-enabled')
220+
return getOptionalBooleanInput('develocity-injection-enabled')
221221
}
222222

223223
getDevelocityUrl(): string {
224224
return core.getInput('develocity-url')
225225
}
226226

227227
getDevelocityAllowUntrustedServer(): boolean | undefined {
228-
return getMaybeBooleanInput('develocity-allow-untrusted-server')
228+
return getOptionalBooleanInput('develocity-allow-untrusted-server')
229229
}
230230

231231
getDevelocityCaptureFileFingerprints(): boolean | undefined {
232-
return getMaybeBooleanInput('develocity-capture-file-fingerprints')
232+
return getOptionalBooleanInput('develocity-capture-file-fingerprints')
233233
}
234234

235235
getDevelocityEnforceUrl(): boolean | undefined {
236-
return getMaybeBooleanInput('develocity-enforce-url')
236+
return getOptionalBooleanInput('develocity-enforce-url')
237237
}
238238

239239
getDevelocityPluginVersion(): string {
@@ -370,14 +370,10 @@ function getBooleanInput(paramName: string, paramDefault = false): boolean {
370370
throw TypeError(`The value '${paramValue} is not valid for '${paramName}. Valid values are: [true, false]`)
371371
}
372372

373-
function getMaybeBooleanInput(paramName: string): boolean | undefined {
373+
function getOptionalBooleanInput(paramName: string): boolean | undefined {
374374
const paramValue = core.getInput(paramName)
375-
switch (paramValue?.toLowerCase().trim()) {
376-
case 'false':
377-
return false
378-
case 'true':
379-
return true
380-
default:
381-
return undefined
375+
if (paramValue === '') {
376+
return undefined
382377
}
378+
return getBooleanInput(paramName)
383379
}

0 commit comments

Comments
 (0)