Skip to content

[Collections] Signing (part 2): signed collections #3241

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

Closed
wants to merge 9 commits into from

Conversation

yim-lee
Copy link
Contributor

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

This is part 2 of a series of PRs to support package collection signing.

Part 1

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.

This is part 1 of a series of PRs to support package collection signing.

Modifications:
- New `PackageCollectionsSigning` module
- Add `Certificate` type
- Add EC and RSA key types
This is part 2 of a series of PRs to support package collection signing.

[Part 1](swiftlang#3238)

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
Copy link
Contributor Author

yim-lee commented Feb 6, 2021

@swift-ci please smoke test

1 similar comment
@yim-lee
Copy link
Contributor Author

yim-lee commented Feb 6, 2021

@swift-ci please smoke test

@yim-lee
Copy link
Contributor Author

yim-lee commented Feb 6, 2021

@swift-ci please smoke test

@tomerd
Copy link
Contributor

tomerd commented Feb 6, 2021

13:17:30 Test Suite 'Selected tests' started at 2021-02-06 21:16:17.512
13:17:30 Test Suite 'SignatureTests' started at 2021-02-06 21:16:17.516
13:17:30 Test Case 'SignatureTests.test_RS256_generateAndValidate_happyCase' started at 2021-02-06 21:16:17.516
13:17:30 
/home/buildnode/jenkins/workspace/swift-package-manager-Linux-smoke-test/branch-main/swiftpm/Tests/PackageCollectionsSigningTests/SignatureTests.swift:20: error: SignatureTests.test_RS256_generateAndValidate_happyCase : failed - bioConversionFailure
13:17:30 Test Case 'SignatureTests.test_RS256_generateAndValidate_happyCase' failed (0.331 seconds)

@yim-lee not sure why it failed on the smoked-test only

@yim-lee
Copy link
Contributor Author

yim-lee commented Feb 6, 2021

@swift-ci please smoke test

@yim-lee not sure why it failed on the smoked-test only

Yeah not sure. Trying to figure out.

@yim-lee
Copy link
Contributor Author

yim-lee commented Feb 6, 2021

@swift-ci please smoke test

@yim-lee
Copy link
Contributor Author

yim-lee commented Feb 7, 2021

@swift-ci please smoke test

@yim-lee
Copy link
Contributor Author

yim-lee commented Feb 7, 2021

@swift-ci please smoke test

@yim-lee
Copy link
Contributor Author

yim-lee commented Feb 7, 2021

I don't see which changes in this PR would cause the CI problems. Will redo it.

@yim-lee yim-lee closed this Feb 7, 2021
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 7, 2021
This is part 2 of a series of PRs to support package collection signing.

Depends on [part 1](swiftlang#3241 (comment))

Modifications:

Add support for signing with EC and RSA keys
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 7, 2021
This is part 2 of a series of PRs to support package collection signing.

Depends on [part 1](swiftlang#3241)

Modifications:

Add support for signing with EC and RSA keys
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 7, 2021
This is part 2 of a series of PRs to support package collection signing.

Depends on [part 1](swiftlang#3241)

Modifications:

Add support for signing with EC and RSA keys
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 8, 2021
This is part 2 of a series of PRs to support package collection signing.

Depends on [part 1](swiftlang#3241)

Modifications:

Add support for signing with EC and RSA keys
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 9, 2021
This is part 2 of a series of PRs to support package collection signing.

Depends on [part 1](swiftlang#3241)

Modifications:

Add support for signing with EC and RSA keys
yim-lee added a commit to yim-lee/swift-package-manager that referenced this pull request Feb 10, 2021
This is part 2 of a series of PRs to support package collection signing.

Depends on [part 1](swiftlang#3241)

Modifications:

Add support for signing with EC and RSA keys
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants