mirror of
https://github.com/jtesta/ssh-audit.git
synced 2024-11-24 11:31:40 +01:00
Added CONTRIBUTING.md (#54).
This commit is contained in:
parent
a7581e07dc
commit
13b065b316
26
CONTRIBUTING.md
Normal file
26
CONTRIBUTING.md
Normal file
@ -0,0 +1,26 @@
|
|||||||
|
# Contributing to ssh-audit
|
||||||
|
|
||||||
|
We are very much open to receiving patches from the community! To encourage participation, passing Travis tests, unit tests, etc., *is OPTIONAL*. As long as the patch works properly, it can be merged (please submit pull requests to the `dev` branch).
|
||||||
|
|
||||||
|
However, if you can submit patches that pass all of our automated tests, then you'll lighten the load for the project maintainer (who already has enough to do!). This document describes what tests are done and what documentation is maintained.
|
||||||
|
|
||||||
|
*Anything extra you can do is appreciated!*
|
||||||
|
|
||||||
|
|
||||||
|
## Tox Tests
|
||||||
|
|
||||||
|
Tox is used to do unit testing, linting with [pylint](http://pylint.pycqa.org/en/latest/) & [flake8](https://flake8.pycqa.org/en/latest/), and static type-checking with [mypy](https://mypy.readthedocs.io/en/stable/).
|
||||||
|
|
||||||
|
Install tox with `apt install tox`, then simply run `tox` in the top-level directory. Look for any error messages in the (verbose) output.
|
||||||
|
|
||||||
|
|
||||||
|
## Docker Tests
|
||||||
|
|
||||||
|
Docker is used to run ssh-audit against various real SSH servers (OpenSSH, Dropbear, and TinySSH). The output is then diff'ed against the expected result. Any differences result in failure.
|
||||||
|
|
||||||
|
The docker tests are run with `./docker_test.sh`. The first time it is run, it will download and compile the SSH servers; this may take awhile. Subsequent runs, however, will take only a minute to complete, as the docker image will already be up-to-date.
|
||||||
|
|
||||||
|
|
||||||
|
## Man Page
|
||||||
|
|
||||||
|
The `ssh-audit.1` man page documents the various features of ssh-audit. If features are added, or significant behavior is modified, the man page needs to be updated.
|
Loading…
Reference in New Issue
Block a user