Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RheinSec (Network Key Storage) Pentest feedback implementation #18

Open
wants to merge 149 commits into
base: main
Choose a base branch
from

Conversation

pl-CJump
Copy link
Contributor

All findings from the Pentest were addressed and fixed.

  • deprecated sodiumoxide crate removed
  • added possibility for user to choose if they want to trust self signed certificates

Phillip Schaefer and others added 30 commits May 13, 2024 11:33
cargo.toml
lib.rs
mod.rs
mod.rs
provider_handle_tests.rs
"99999" muss durch die echten nks_adress und nks_token ersetzt werden.
"99999" muss durch die echten nks_adress und nks_token ersetzt werden.
…ic errors, added initial impl for nks config
original error was
error[E0277]: the trait bound &std::option::Option<Url>: IntoUrl is not satisfied --> src/nks/hcvault/key_handle.rs:202:19 | 202 | .post(&self.nks_address) | ---- ^^^^^^^^^^^^^^^^^ the trait IntoUrl is not implemented for &std::option::Option<Url> | | | required by a bound introduced by this call | = help: the following other types implement trait IntoUrl: Url std::string::String &'a str &'a std::string::String
pl-CJump and others added 28 commits June 8, 2024 12:01
…rust invalid certificates via an environment variable
… doku and deleted Cargo_old.lock since its unused
@pl-CJump pl-CJump marked this pull request as draft June 17, 2024 11:57
@pl-CJump pl-CJump marked this pull request as ready for review June 17, 2024 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants