Add skipFirewallCheck to IActionInputs

Azure · Jun 26, 2024 · 6b02c1f · 6b02c1f
1 parent e928598
commit 6b02c1f
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 7 deletions.
diff --git a/lib/main.js b/lib/main.js
diff --git a/src/AzureSqlAction.ts b/src/AzureSqlAction.ts
@@ -19,6 +19,7 @@ export interface IActionInputs {
     connectionConfig: SqlConnectionConfig;
     filePath: string;
     additionalArguments?: string;
+    skipFirewallCheck: boolean;
 }
 
 export interface IDacpacActionInputs extends IActionInputs {

diff --git a/src/main.ts b/src/main.ts
@@ -25,7 +25,7 @@ export default async function run() {
         const azureSqlAction = new AzureSqlAction(inputs);
 
         // Unless skip-firewall-check is set to true, check if the runner's IP address is allowed to connect to the server
-        if (core.getBooleanInput('skip-firewall-check') !== true) {
+        if (inputs.skipFirewallCheck !== true) {
             const runnerIPAddress = await SqlUtils.detectIPAddress(inputs.connectionConfig);
             if (runnerIPAddress) {
                 let azureResourceAuthorizer = await AuthorizerFactory.getAuthorizer();
@@ -75,7 +75,8 @@ function getInputs(): IActionInputs {
                 actionType: ActionType.SqlAction,
                 connectionConfig: connectionConfig,
                 filePath: filePath,
-                additionalArguments: core.getInput('arguments') || undefined
+                additionalArguments: core.getInput('arguments') || undefined,
+                skipFirewallCheck: core.getBooleanInput('skip-firewall-check')
             };
 
         case Constants.dacpacExtension:
@@ -88,7 +89,8 @@ function getInputs(): IActionInputs {
                 connectionConfig: connectionConfig,
                 filePath: filePath,
                 sqlpackageAction: AzureSqlActionHelper.getSqlpackageActionTypeFromString(action),
-                additionalArguments: core.getInput('arguments') || undefined
+                additionalArguments: core.getInput('arguments') || undefined,
+                skipFirewallCheck: core.getBooleanInput('skip-firewall-check')
             } as IDacpacActionInputs;
 
         case Constants.sqlprojExtension:
@@ -102,11 +104,10 @@ function getInputs(): IActionInputs {
                 filePath: filePath,
                 buildArguments: core.getInput('build-arguments') || undefined,
                 sqlpackageAction: AzureSqlActionHelper.getSqlpackageActionTypeFromString(action),
-                additionalArguments: core.getInput('arguments') || undefined
+                additionalArguments: core.getInput('arguments') || undefined,
+                skipFirewallCheck: core.getBooleanInput('skip-firewall-check')
             } as IBuildAndPublishInputs;
 
-            break;
-
         default:
             throw new Error(`Invalid file type provided as input ${filePath}. File must be a .sql, .dacpac, or .sqlproj file.`)
     }