From 8c5b6ad163e21225bfabea87e854d8eeaa5fc0f9 Mon Sep 17 00:00:00 2001 From: Mikey Lombardi Date: Thu, 31 Jul 2025 09:08:37 -0500 Subject: [PATCH] (GHA) Fix authorization workflow Prior to this change, the authorization workflow used the PowerShell script parameter names and casing for the action parameter names. This worked for all previous parameters because PowerShell is case insensitive and the parameters were only one word. However, in #12181, we added the `authorized_accounts` parameter to the workflow. For GitHub Actions, we use `snake_case` for parameter names, where PowerShell best practice is to use `PascalCase`. PowerShell's case insensitivity meant that we were able to handle the prior parameters, which the workflow sends to the handler scripts as the `INPUT_` environment variables. When we added `authorized_accounts`, the handler scripts were looking for `INPUT_AuthorizedAccounts`, which doesn't exist. This change updates the `Name` key for every defined parameter in the `verification/authorization/v1` workflow to match the actual workflow parameter name and casing. --- .../verification/authorization/v1/Parameters.psd1 | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/.github/actions/verification/authorization/v1/Parameters.psd1 b/.github/actions/verification/authorization/v1/Parameters.psd1 index 306e47874109..4205303dd399 100644 --- a/.github/actions/verification/authorization/v1/Parameters.psd1 +++ b/.github/actions/verification/authorization/v1/Parameters.psd1 @@ -1,7 +1,7 @@ @{ Parameters = @( @{ - Name = 'Repository' + Name = 'repository' Type = 'string' IfNullOrEmpty = { param($ErrorTarget) @@ -30,7 +30,7 @@ } } @{ - Name = 'AuthorizedAccounts' + Name = 'authorized_accounts' Type = 'String[]' IfNullOrEmpty = { param($ErrorTarget) @@ -50,7 +50,7 @@ } @{ - Name = 'Permissions' + Name = 'permissions' Type = 'String[]' IfNullOrEmpty = { param($ErrorTarget) @@ -118,7 +118,7 @@ } @{ - Name = 'Target' + Name = 'target' Type = 'String[]' IfNullOrEmpty = { param($ErrorTarget) @@ -191,7 +191,7 @@ } @{ - Name = 'User' + Name = 'user' Type = 'String' IfNullOrEmpty = { param($ErrorTarget) @@ -222,4 +222,4 @@ } } ) -} \ No newline at end of file +}