Skip to content
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

[WIP] Update dependencies. #130

Closed
wants to merge 3 commits into from
Closed

[WIP] Update dependencies. #130

wants to merge 3 commits into from

Conversation

stephank
Copy link
Member

@stephank stephank commented Jan 22, 2017

Notable upgrades are Hyper 0.10, Iron 0.5 and the openssl crate 0.9, among
smaller dep upgrades. Only the openssl crate upgrade required code changes.

Waiting on:

@stephank stephank force-pushed the featureDeps branch 5 times, most recently from f63bb83 to 103692e Compare February 1, 2017 09:25
@stephank
Copy link
Member Author

stephank commented Feb 1, 2017

I've created a second branch where I've completely replaced OpenSSL with hyper-native-tls and ring. It'll only be used as a dependency of hyper-native-tls on Linux.

https://github.com/portier/portier-broker/compare/featureDeps2

It's almost finished, but requires some changes in ring too:

  • Extract RSAKeyPair public components so we can build a JWK
  • Use a JWK to verify a signature

@stephank stephank force-pushed the featureDeps branch 2 times, most recently from 04a9318 to 820316e Compare February 4, 2017 14:15
@stephank
Copy link
Member Author

stephank commented Feb 4, 2017

Alright, I've now pushed the ring-based branch to this PR! It depends on my in-progress branch of ring, but is working nicely. :)

Once mature, we may even be able to switch to rustls. :)

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 81ae8ae on featureDeps into ** on master**.

Stéphan Kochen added 3 commits February 23, 2017 17:03
Notable upgrades are Hyper 0.10, Iron 0.5. We now use `hyper-native-tls` and
`ring` instead of openssl directly, dropping openssl completely on platforms
other than Linux. The code changes are all related to dropping openssl.
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 5f71583 on featureDeps into ** on master**.

@stephank
Copy link
Member Author

stephank commented Aug 7, 2017

I'm closing this, because I'm currently not motivated to push along the upstream changes, which have bitrotten, just as this branch.

I'll clean up the branches in a bit. :)

@stephank stephank closed this Aug 7, 2017
@stephank stephank deleted the featureDeps branch August 7, 2017 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants