Skip to content
This repository has been archived by the owner on May 11, 2023. It is now read-only.

Add chore cargo-deny #228

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

pinkforest
Copy link
Contributor

@pinkforest pinkforest commented Aug 24, 2022

Signed-off-by: MissM [email protected]

Cargo-deny action by the lovely Embark crowd can be used to note on advisories / licenses upon PR / push

I've set so any advisory will not trash CI completely suddenly.

We can have separate audit job that runs periodically and raises Issues automatically upon any new advisory

EDIT: the deny action obviously fails atm

Licensing clarifications needed - besides the ones that FSF has decided on already as copyleft / compatible:

Advisories - Security / Errors

We can either deal with or supress them after evaluating impact if any -

RUSTSEC-2020-0043 - DoS - Insufficient size checks in outgoing buffer in ws allows remote attacker to run the process OOM
Crate ws is brought by walletconnect we have no control of - direct fork exists as parity-ws - housleyjk/ws-rs#291

RUSTSEC-2020-0159, RUSTSEC-2020-0071 - #231 - clarify & bump chrono features/time dependency

Advisories - Informational / Warns

Usually we nudge people to move away from these and may contain unsound / other concerns but no concrete threats

Unmaintained cbor, ansi_term, ...

Signed-off-by: pinkforest <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant