Skip to content

[match] Have Match download Apple's Developer ID intermediate certificates#21364

Closed
iOSGeekster wants to merge 2 commits into
fastlane:masterfrom
iOSGeekster:developer-id-certs
Closed

[match] Have Match download Apple's Developer ID intermediate certificates#21364
iOSGeekster wants to merge 2 commits into
fastlane:masterfrom
iOSGeekster:developer-id-certs

Conversation

@iOSGeekster

Copy link
Copy Markdown
Contributor

Checklist

  • I've run bundle exec rspec from the root directory to see all new and existing tests pass
  • I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
  • I see several green ci/circleci builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)
  • I've read the Contribution Guidelines
  • I've updated the documentation if necessary.

Motivation and Context

When using match to sign a Mac app, that is to be published via a notarized dmg file, a Developer ID certificate is used to sign the app.
And just like when signing apps for the AppStore, with distribution certificates, a Apple WWDR certificate is needed in the keychain to validate the certificate chain.

So Apple's two developer ID intermediate certificates, from Apples Certificate Authority Page has been added to the existing list of WWDR certificates

Description

  • Added the two Developer ID certificates to the list of WWDR certificates in cert_checker.rb
  • Added specs to verify installation of said certificates

@iOSGeekster iOSGeekster marked this pull request as draft June 30, 2023 09:49
@iOSGeekster iOSGeekster marked this pull request as ready for review June 30, 2023 09:49
@iOSGeekster iOSGeekster changed the title Have Match download Apple's Developer ID intermediate certificates [match] Have Match download Apple's Developer ID intermediate certificates Jul 3, 2023
@iOSGeekster

Copy link
Copy Markdown
Contributor Author

Is it someway possible to restart the checks, without authorising CircleCI? I think that failing tests are just a bit flaky.

@AliSoftware

AliSoftware commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

@iOSGeekster Thanks a lot for having submitted this PR ❤️ . I know it's been submitted a long time ago, so sorry for not getting to it sooner 😓

We've opened #29890 to rebase the changes you did in this PR on top of the latest commits and have merged the changes from that PR since, hence yours being auto-closed now that the same changes have been applied by the other one.

@github-actions

github-actions Bot commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

The pull request #29890 that closed this issue was merged and released as part of fastlane 2.232.0 🚀
Please let us know if the functionality works as expected as a reply here. If it does not, please open a new issue. Thanks!

@iOSGeekster

Copy link
Copy Markdown
Contributor Author

@iOSGeekster Thanks a lot for having submitted this PR ❤️ . I know it's been submitted a long time ago, so sorry for not getting to it sooner 😓

We've opened #29890 to rebase the changes you did in this PR on top of the latest commits and have merged the changes from that PR since, hence yours being auto-closed now that the same changes have been applied by the other one.

No worries. Havn't pushed much against this getting looked at. Glad to see that the changes were usable.

@github-actions github-actions Bot locked and limited conversation to collaborators Apr 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants