Skip to content

Commit

Permalink
fix(mac): signing cert filter incorrectly selects certificates (#6094) (
Browse files Browse the repository at this point in the history
#6101) (#6105)

* fix(mac): Adding Developer ID Application entry for development signing when not mas (#6094)
* fix(mac): Removing 3rd Party Mac Developer Application certificate selector (#6101)
  • Loading branch information
mmaietta committed Jul 28, 2021
1 parent 7e0a77a commit 4a177dc
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
6 changes: 6 additions & 0 deletions .changeset/great-pumas-trade.md
@@ -0,0 +1,6 @@
---
"app-builder-lib": patch
---

fix(mac): Adding Developer ID Application entry for development signing when building not a mas target. Fixes: #6094
fix(mac): Removing 3rd Party Mac Developer Application certificate selector. Fixes: #6101
4 changes: 2 additions & 2 deletions packages/app-builder-lib/src/macPackager.ts
Expand Up @@ -453,7 +453,7 @@ export default class MacPackager extends PlatformPackager<MacConfiguration> {

function getCertificateTypes(isMas: boolean, isDevelopment: boolean): CertType[] {
if (isDevelopment) {
return ["Mac Developer", "Apple Development"]
return isMas ? ["Mac Developer", "Apple Development"] : ["Developer ID Application"]
}
return isMas ? ["3rd Party Mac Developer Application", "Apple Distribution"] : ["Developer ID Application"]

This comment has been minimized.

Copy link
@UnsignedInt8

UnsignedInt8 Aug 3, 2021

No!!! 3rd Party Mac Developer Application should not be removed. Please add it back.

return isMas ? ["Apple Distribution"] : ["Developer ID Application"]
}

0 comments on commit 4a177dc

Please sign in to comment.