quininer / tokio-rustls

Asynchronous TLS/SSL streams for Tokio using Rustls.
142 stars 38 forks source link

Clarify the license #29

Closed erickt closed 5 years ago

erickt commented 5 years ago

Hello,

The description of the license in the README as added in #1 by @briansmith appears to be vague:

tokio-rustls is primarily distributed under the terms of both the MIT license and the Apache License (Version 2.0), with portions covered by various BSD-like licenses.

It appears @briansmith may have got this phrase from some other rust projects, like futures-rs. They have since changed to using this phrasing:

# License
This project is licensed under either of

Apache License, Version 2.0, (LICENSE-APACHE or http://www.apache.org/licenses/LICENSE-2.0)
MIT license (LICENSE-MIT or http://opensource.org/licenses/MIT)
at your option.

### Contribution
Unless you explicitly state otherwise, any contribution intentionally submitted for inclusion in futures-rs by you, as defined in the Apache-2.0 license, shall be dual licensed as above, without any additional terms or conditions.

Would that be more along the lines of what you intended?

erickt commented 5 years ago

For context, here's the commit in futures-rs that explains why they switched the phrasing: https://github.com/rust-lang-nursery/futures-rs/commit/2d95ac92c46eb1f287bffe21f164362661fd0c24.

quininer commented 5 years ago

The new phrasing seems more reasonable. Would you like to submit a PR?