Skip to content

Add macos signing for package files#24015

Merged
TravisEz13 merged 9 commits into
PowerShell:masterfrom
adityapatwardhan:obp-test-macsigning
Jul 11, 2024
Merged

Add macos signing for package files#24015
TravisEz13 merged 9 commits into
PowerShell:masterfrom
adityapatwardhan:obp-test-macsigning

Conversation

@adityapatwardhan
Copy link
Copy Markdown
Member

PR Summary

Add signing for macos package files

PR Context

PR Checklist

Comment on lines +178 to +183
ConnectedServiceName: 'ESRPMacOSSigning'
AppRegistrationClientId: '9dfad4b0-b54d-463e-a4df-340b56fa720b'
AppRegistrationTenantId: '72f988bf-86f1-41af-91ab-2d7cd011db47'
AuthAKVName: 'pwsh-CICD-Keyvault'
AuthCertName: 'PS-macos-signing'
AuthSignCertName: 'ESRP-OneCert' # this is not needed for pkg signing
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

offline discussion

signConfigType: inlineSignParams
inlineOperation: |
[{
"KeyCode": "CP-401337-Apple",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

offline discussion

"ToolVersion": "1.0"
}]
SessionTimeout: 90
ServiceEndpointUrl: 'https://api.esrp.microsoft.com/api/v2'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

offline discussion

@adityapatwardhan
Copy link
Copy Markdown
Member Author

@TravisEz13 - updated

@TravisEz13 TravisEz13 merged commit 390a341 into PowerShell:master Jul 11, 2024
@microsoft-github-policy-service
Copy link
Copy Markdown
Contributor

microsoft-github-policy-service Bot commented Jul 11, 2024

📣 Hey @adityapatwardhan, how did we do? We would love to hear your feedback with the link below! 🗣️

🔗 https://aka.ms/PSRepoFeedback

adityapatwardhan added a commit that referenced this pull request Jul 15, 2024
* Sign macos packages

* Fix typo

* Add variables

* Remove signing cert

* Update signing

* Add signing cert

* Add uncompress

* Add variable group

* Download signed macos packages
adityapatwardhan added a commit that referenced this pull request Jul 15, 2024
* Sign macos packages

* Fix typo

* Add variables

* Remove signing cert

* Update signing

* Add signing cert

* Add uncompress

* Add variable group

* Download signed macos packages
@SeeminglyScience SeeminglyScience added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Aug 20, 2024
chrisdent-de pushed a commit to chrisdent-de/PowerShell that referenced this pull request Sep 12, 2024
* Sign macos packages

* Fix typo

* Add variables

* Remove signing cert

* Update signing

* Add signing cert

* Add uncompress

* Add variable group

* Download signed macos packages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport-7.2.x-Done Backport-7.4.x-Done CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants