From 5f768b3673f8b3ecdf24c8e995b8f7a359ec0dfe Mon Sep 17 00:00:00 2001
From: Ben Broderick Phillips <bebroder@microsoft.com>
Date: Fri, 25 Aug 2023 14:38:28 -0400
Subject: [PATCH] Skip strong name signing validation for secret cli

---
 .../templates/stages/archetype-sdk-tool-dotnet.yml   | 12 ++++++++----
 tools/secret-management/ci.yml                       |  2 ++
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/eng/pipelines/templates/stages/archetype-sdk-tool-dotnet.yml b/eng/pipelines/templates/stages/archetype-sdk-tool-dotnet.yml
index 0522617ee2f..7c63106ced3 100644
--- a/eng/pipelines/templates/stages/archetype-sdk-tool-dotnet.yml
+++ b/eng/pipelines/templates/stages/archetype-sdk-tool-dotnet.yml
@@ -42,6 +42,9 @@ parameters:
   - name: ShouldPublishSymbols
     type: boolean
     default: true
+  - name: RequireStrongNames
+    type: boolean
+    default: true
 
 
 variables:
@@ -79,8 +82,8 @@ stages:
               DOTNET_MULTILEVEL_LOOKUP: 0
 
           # This step creates "$(packagesToPublishDir)" directory if it doesn't exist.
-          # This step is necessary since migration to net6.0. This is because since net6.0, 
-          # in case the "Build and Package" above would not output any packages to this directory, 
+          # This step is necessary since migration to net6.0. This is because since net6.0,
+          # in case the "Build and Package" above would not output any packages to this directory,
           # the "Publish to packages artifact" step below would fail on missing directory.
           - pwsh: |
               if (!(Test-Path -PathType container "$(packagesToPublishDir)")) {
@@ -155,9 +158,9 @@ stages:
               DOTNET_SKIP_FIRST_TIME_EXPERIENCE: 1
               DOTNET_CLI_TELEMETRY_OPTOUT: 1
               DOTNET_MULTILEVEL_LOOKUP: 0
-          
+
           - ${{ parameters.TestPostSteps }}
-          
+
           - task: PublishTestResults@2
             condition: succeededOrFailed()
             inputs:
@@ -181,6 +184,7 @@ stages:
         ExeMatrix: ${{ parameters.StandaloneExeMatrix }}
         ShouldPublishExecutables: ${{ parameters.ReleaseBinaries }}
         ShouldPublishSymbols: ${{ parameters.ShouldPublishSymbols }}
+        RequireStrongNames: ${{ parameters.RequireStrongNames }}
 
   - ${{if and(not(eq(length(parameters.DockerDeployments), 0)), ne(variables['Build.Reason'], 'PullRequest'), eq(variables['System.TeamProject'], 'internal'))}}:
     - stage: PublishDockerImages
diff --git a/tools/secret-management/ci.yml b/tools/secret-management/ci.yml
index 764ec1914d8..e1fd3c72137 100644
--- a/tools/secret-management/ci.yml
+++ b/tools/secret-management/ci.yml
@@ -25,3 +25,5 @@ extends:
   template: /eng/pipelines/templates/stages/archetype-sdk-tool-dotnet.yml
   parameters:
     ToolDirectory: tools/secret-management
+    # Sodium.Core dependency is not strong name signed. Skip validation in order to sign secret cli.
+    RequireStrongNames: false