remove unused dependency on `term`

tomaka / rouille

Web framework in Rust

Apache License 2.0

1.12k stars 106 forks source link

remove unused dependency on `term` #224

Closed soruh closed 3 years ago

soruh commented 4 years ago

This PR removes the unused dependency on term which is no longer maintained and has a RUSTSEC advisory. Having the term dependency causes warnings when running cargo audit in reverse dependencies.

MightyPork commented 3 years ago

@bradfier this and #231 seem safe to merge despite the travis error, I just tried it. Could you look into it when you have time?

Rouille now drags 2 years old versions of dependencies into our project, I'm trying to reduce the attack surface using cargo deny...

bradfier commented 3 years ago

It seems quite reasonable indeed, I can't modify the original PR branch but I'll rebase this commit onto master and run the new test suites.

bradfier commented 3 years ago

Merged as 45f1631