-
Notifications
You must be signed in to change notification settings - Fork 56
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
3 changed files
with
107 additions
and
4 deletions.
There are no files selected for viewing
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,51 @@ | ||
# Contributing | ||
|
||
Thanks for considering helping this project. There are many | ||
ways you can help: using the library and reporting bugs, | ||
reporting usability issues, making additions and improvements | ||
to the library, documentation and finding security bugs. | ||
|
||
## Reporting bugs | ||
|
||
Please file a github issue. Include as much information as | ||
possible. Suspected protocol bugs are easier debugged with | ||
a pcap or reproduction steps. | ||
|
||
Feel free to file github issues to get help, or ask a question. | ||
|
||
## Code changes | ||
|
||
Some ideas and guidelines for contributions: | ||
|
||
- For large features, file an issue prior to starting work. | ||
This means everyone can see what is in progress prior to a PR. | ||
- Feel free to submit a PR even if the work is not totally finished, | ||
for feedback or to hand-over. | ||
- Prefer not to reference github issue or PR numbers in commits. | ||
- Try to keep code formatting commits separate from functional commits. | ||
- I run `cargo outdated` prior to major releases; but PRs to update specific | ||
dependencies are welcome. | ||
|
||
## Security bugs | ||
|
||
Please report security bugs by filing a github issue, or by | ||
email to [email protected] if you want to disclose privately. I'll then: | ||
|
||
- Prepare a fix and regression tests. | ||
- Backport the fix and make a patch release for most recent release. | ||
- Submit an advisory to [rustsec/advisory-db](https://github.com/RustSec/advisory-db). | ||
- Refer to the advisory on the main README.md and release notes. | ||
|
||
## Testing | ||
|
||
- Features involving additions to the public API should have (at least) | ||
API-level tests (see [`tests/api.rs`](tests/api.rs)). | ||
- Protocol additions should have some coverage -- consider enabling | ||
corresponding tests in the bogo suite, or writing some adhoc tests. | ||
|
||
PRs which cause test failures or a significant coverage decrease | ||
are unlikely to be accepted. | ||
|
||
## Licensing | ||
|
||
Contributions are made under [rustls-native-certs's licenses](LICENSE). |
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,8 +1,14 @@ | ||
[package] | ||
name = "rustls-native-certs" | ||
version = "0.1.0" | ||
authors = ["Joseph Birr-Pixton <[email protected]>"] | ||
edition = "2018" | ||
authors = ["Joseph Birr-Pixton <[email protected]>"] | ||
license = "Apache-2.0/ISC/MIT" | ||
readme = "README.md" | ||
description = "rustls-native-certs allows rustls to use the platform native certificate store" | ||
homepage = "https://github.com/ctz/rustls-native-certs" | ||
repository = "https://github.com/ctz/rustls-native-certs" | ||
categories = ["network-programming", "cryptography"] | ||
|
||
[dependencies] | ||
rustls = "0.16.0" | ||
|
This file contains 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