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

Add cargo features "client" and "server" #23

Merged
merged 1 commit into from Jul 15, 2020
Merged

Conversation

mbrubeck
Copy link
Contributor

@mbrubeck mbrubeck commented May 24, 2020

Since many consumers of this library do not need both client and server features, this allows them to shrink their dependencies somewhat.

In particular, server applications would no longer need to build the unused webpki and webpki-roots crate.

I would also suggest making these features disabled by default in a future version, to increase the likeliness that consumers will choose the features they actually need. (This would be a breaking change.)

@mbrubeck
Copy link
Contributor Author

test_0rtt is failing on Travis. I can reproduce this failure locally both on this branch and on master, so I don't think it's related to my changes.

Since many consumers of this library do not need both client and server
features, this allows them to shrink their dependencies somewhat.
@skade
Copy link
Collaborator

skade commented Jun 4, 2020

Seems unrelated. I'm going to check that locally.

@mbrubeck
Copy link
Contributor Author

mbrubeck commented Jun 6, 2020

In particular, server applications would no longer need to build the unused webpki and webpki-roots crates.

This was incorrect. Because rustls depends on webpki, server applications would still need to build that crate, though they would no longer need to build webpki-roots.

@skade skade merged commit dc30039 into async-rs:master Jul 15, 2020
@skade
Copy link
Collaborator

skade commented Jul 15, 2020

Added documentation to the README and merged. Thanks @mbrubeck !

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.

None yet

2 participants