Skip to content

[Collections] Signing (apple, 3): certificate validations #3269

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 18, 2021

Conversation

yim-lee
Copy link
Contributor

@yim-lee yim-lee commented Feb 13, 2021

This is part 3 of a series of PRs to support package collection signing on Apple platforms. Originally #3245.

Depends on #3264

Modifications:

  • Add certificate policies to validate collection-signing certificates.
  • Wire up PackageCollectionSigning with certificate policies.

yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 13, 2021
This is part 3 of a series of PRs to support package collection signing on **non-Apple** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 13, 2021
This is part 3 of a series of PRs to support package collection signing on **non-Apple** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 13, 2021
This is part 4 of a series of PRs to support package collection signing on Apple platforms. Originally swiftlang#3252.

Depends on  swiftlang#3269.

Modifications:
- Add `PackageCollectionsSigning` dependency to `PackageCollections` module
- Verify collection signature in `JSONPackageCollectionProvider`
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 13, 2021
This is part 4 of a series of PRs to support package collection signing on Apple platforms. Originally swiftlang#3252.

Depends on  swiftlang#3269.

Modifications:
- Add `PackageCollectionsSigning` dependency to `PackageCollections` module
- Verify collection signature in `JSONPackageCollectionProvider`
@yim-lee
Copy link
Contributor Author

yim-lee commented Feb 16, 2021

cc @catmurdoch

@tomerd tomerd added the ready Author believes the PR is ready to be merged & any feedback has been addressed label Feb 16, 2021
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 17, 2021
This is part 4 of a series of PRs to support package collection signing on Apple platforms. Originally swiftlang#3252.

Depends on  swiftlang#3269.

Modifications:
- Add `PackageCollectionsSigning` dependency to `PackageCollections` module
- Verify collection signature in `JSONPackageCollectionProvider`
This is part 3 of a series of PRs to support package collection signing on **Apple** platforms. Originally swiftlang#3245.

Depends on swiftlang#3264

Modifications:

- Add certificate policies to validate collection-signing certificates.
- Wire up `PackageCollectionSigning` with certificate policies.
@yim-lee
Copy link
Contributor Author

yim-lee commented Feb 18, 2021

@swift-ci please smoke test

yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 18, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 18, 2021
This is part 4 of a series of PRs to support package collection signing on **Apple** platforms. Originally swiftlang#3252.

Depends on  swiftlang#3269.

Modifications:
- Add `PackageCollectionsSigning` dependency to `PackageCollections` module
- Verify collection signature in `JSONPackageCollectionProvider`
@yim-lee yim-lee merged commit fd14a63 into swiftlang:main Feb 18, 2021
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 18, 2021
This is part 4 of a series of PRs to support package collection signing on **Apple** platforms. Originally swiftlang#3252.

Depends on  swiftlang#3269.

Modifications:
- Add `PackageCollectionsSigning` dependency to `PackageCollections` module
- Verify collection signature in `JSONPackageCollectionProvider`
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 18, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 18, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 18, 2021
This is part 4 of a series of PRs to support package collection signing on **Apple** platforms. Originally swiftlang#3252.

Depends on  swiftlang#3269.

Modifications:
- Add `PackageCollectionsSigning` dependency to `PackageCollections` module
- Verify collection signature in `JSONPackageCollectionProvider`
yim-lee added a commit that referenced this pull request Feb 19, 2021
This is part 4 of a series of PRs to support package collection signing on **Apple** platforms. Originally #3252.

Depends on  #3269.

Modifications:
- Add `PackageCollectionsSigning` dependency to `PackageCollections` module
- Verify collection signature in `JSONPackageCollectionProvider`
- Don't cache CertificatePolicy
- Add support for additionalTrustedRootCerts
- Custom certificate policy by collection source
- Throw cannotVerifySignature when no trusted roots configured
- Feature flag
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 20, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 22, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 26, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 26, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Mar 2, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Mar 9, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Mar 10, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Mar 11, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Mar 11, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Mar 12, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Mar 12, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Mar 13, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Mar 20, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Mar 22, 2021
This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3245.

Depends on swiftlang#3265, swiftlang#3269
yim-lee added a commit that referenced this pull request Mar 24, 2021
* [Collections] Signing (all, 1): certificate and key types

This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally #3238.

Depends on #3259

* [Collections] Signing (all, 2): signed collections

This is part 2 of a series of PRs to support package collection signing on **all** platforms. Originally #3242.

Depends on #3260, #3264

* [Collections] Signing (all, 3): certificate validations

This is part 3 of a series of PRs to support package collection signing on **all** platforms. Originally #3245.

Depends on #3265, #3269

* [Collections] Signing (all, 3.b): OCSP support for non-Apple platforms

This is part 3.b of a series of PRs to support package collection signing on **all** platforms.

Depends on #3270

Modifications:
- Add `PackageCollectionsSigningLibc` module
- Add OCSP support for non-Apple platforms through `PackageCollectionsSigningLibc`

* Use shared callbackQueue and diagnosticsEngine in tests

* Update platform check

* Better OCSP error handling and cache OCSP results

* Don't default to BoringSSL

* OpenSSL license

* fixup

* Use ptr.count

* Don't blindly drop X509_V_ERR_UNHANDLED_CRITICAL_EXTENSION error

* Add withUnsafeMutablePointer for accessing underlying X509 pointer

* Fix leaks

* Use withExtendedLifetime

* OSCP response handling

* Delete remove from cache call

* OCSP should fail open

* Verify OCSP response

* Create new X509_STORE per invocation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready Author believes the PR is ready to be merged & any feedback has been addressed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants