[Collections] Signing (apple, 1): certificate and key types #3259
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
yim-lee
requested review from
abertelrud,
friedbunny,
neonichu and
tomerd
as code owners
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 12, 2021
This is part 1 of a series of PRs to support package collection signing on **non-Apple** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
This was referenced Feb 12, 2021
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 12, 2021
This is part 2 of a series of PRs to support package collection signing on **Apple** platforms. Originally swiftlang#3242. Depends on [part 1](swiftlang#3259) Modifications: - Add support for signing with EC and RSA keys - Add `Signature`, which is similar to JWS - Introduce `CertificatePolicy` protocol (more on this in part 3) - `PackageCollectionsSigning` - given a package collection, certificate and its private key, generate a "signed" collection; and reverse that process to validate collection signature.
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 12, 2021
This is part 2 of a series of PRs to support package collection signing on **Apple** platforms. Originally swiftlang#3242. Depends on [part 1](swiftlang#3259) Modifications: - Add support for signing with EC and RSA keys - Add `Signature`, which is similar to JWS - Introduce `CertificatePolicy` protocol (more on this in part 3) - `PackageCollectionsSigning` - given a package collection, certificate and its private key, generate a "signed" collection; and reverse that process to validate collection signature.
tomerd
approved these changes
Feb 12, 2021
tomerd
reviewed
Feb 12, 2021
| @@ -152,11 +152,16 @@ let package = Package( | |||
| /** Package collections models */ | |||
| name: "PackageCollectionsModel", | |||
| dependencies: []), | |||
|
|
|||
| .target( | |||
tomerd
reviewed
Feb 12, 2021
Sources/PackageCollectionsSigning/Certificate/Certificate.swift
Outdated
Show resolved
Hide resolved
tomerd
reviewed
Feb 12, 2021
Sources/PackageCollectionsSigning/Certificate/Certificate.swift
Outdated
Show resolved
Hide resolved
tomerd
reviewed
Feb 12, 2021
Sources/PackageCollectionsSigning/Certificate/Certificate.swift
Outdated
Show resolved
Hide resolved
tomerd
reviewed
Feb 12, 2021
|
|
||
| //===----------------------------------------------------------------------===// | ||
| // | ||
| // This source file is part of the SwiftCrypto open source project |
There was a problem hiding this comment.
its unfortunate that we cannot use SwiftCrypto types here instead of re-defining them, is this something @Lukasa can help fix/expose in SwiftCrypto?
There was a problem hiding this comment.
At this time the answer is no: the ASN.1 code is changing rapidly. While I do want to build out Swift ASN.1 infrastructure, until we get something approaching complete coverage of ASN.1 there's essentially no guarantees on stability (indeed, I plan to break it soon).
In the short term copy-pasting is fine. Longer term when we resolve the ASN.1 issue we should be taking an item to remove these files.
There was a problem hiding this comment.
Actually they can be removed sooner than then: as soon as SwiftPM can drop support for Catalina we no longer need these.
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 13, 2021
This is part 1 of a series of PRs to support package collection signing on **non-Apple** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 13, 2021
This is part 2 of a series of PRs to support package collection signing on **Apple** platforms. Originally swiftlang#3242. Depends on [part 1](swiftlang#3259) Modifications: - Add support for signing with EC and RSA keys - Add `Signature`, which is similar to JWS - Introduce `CertificatePolicy` protocol (more on this in part 3) - `PackageCollectionsSigning` - given a package collection, certificate and its private key, generate a "signed" collection; and reverse that process to validate collection signature.
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 13, 2021
This is part 2 of a series of PRs to support package collection signing on **Apple** platforms. Originally swiftlang#3242. Depends on [part 1](swiftlang#3259) Modifications: - Add support for signing with EC and RSA keys - Add `Signature`, which is similar to JWS - Introduce `CertificatePolicy` protocol (more on this in part 3) - `PackageCollectionsSigning` - given a package collection, certificate and its private key, generate a "signed" collection; and reverse that process to validate collection signature.
Lukasa
approved these changes
Feb 16, 2021
|
cc @catmurdoch |
tomerd
added
the
ready
This is part 1 of a series of PRs to support package collection signing on **Apple** platforms. Originally swiftlang#3238. Modifications: - New `PackageCollectionsSigning` module - Add `Certificate` type - Add EC and RSA key types
|
@swift-ci please smoke test |
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 17, 2021
This is part 2 of a series of PRs to support package collection signing on **Apple** platforms. Originally swiftlang#3242. Depends on [part 1](swiftlang#3259) Modifications: - Add support for signing with EC and RSA keys - Add `Signature`, which is similar to JWS - Introduce `CertificatePolicy` protocol (more on this in part 3) - `PackageCollectionsSigning` - given a package collection, certificate and its private key, generate a "signed" collection; and reverse that process to validate collection signature.
yim-lee
added a commit
that referenced
this pull request
Feb 18, 2021
This is part 2 of a series of PRs to support package collection signing on **Apple** platforms. Originally #3242. Depends on [part 1](#3259) Modifications: - Add support for signing with EC and RSA keys - Add `Signature`, which is similar to JWS - Introduce `CertificatePolicy` protocol (more on this in part 3) - `PackageCollectionsSigning` - given a package collection, certificate and its private key, generate a "signed" collection; and reverse that process to validate collection signature.
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 18, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 18, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 18, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 18, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 20, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 22, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 26, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Feb 26, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Mar 2, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Mar 9, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Mar 10, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Mar 11, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Mar 11, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Mar 12, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Mar 12, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Mar 13, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Mar 20, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
yim-lee
added a commit
to yim-lee/swift-package-manager
that referenced
this pull request
Mar 22, 2021
This is part 1 of a series of PRs to support package collection signing on **all** platforms. Originally swiftlang#3238. Depends on swiftlang#3259
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is part 1 of a series of PRs to support package collection signing on Apple platforms. Originally
#3238.
Modifications:
PackageCollectionsSigningmoduleCertificatetype