Skip to content

Commit

Permalink
Update dependencies and don't zero out uri_acc as this only contains …
Browse files Browse the repository at this point in the history
…5 characters of the SHA1 hash.
  • Loading branch information
brycx committed Jun 2, 2019
1 parent f83aa43 commit 7e0509c
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
8 changes: 4 additions & 4 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,12 @@ repository = "https://github.com/brycx/checkpwn"
license = "MIT"

[dependencies]
colored = "1.7"
colored = "1.8"
sha-1 = { version = "0.8.1", default-features = false }
hex = "0.3.2"
reqwest = "0.9.11"
rpassword = "3.0.1"
reqwest = "0.9.17"
rpassword = "3.0.2"
zeroize = "0.8.0"

[dev-dependencies]
assert_cmd = "0.11.0"
assert_cmd = "0.11.1"
3 changes: 1 addition & 2 deletions src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ fn pass_check(data_search: &api::PassArg) {
.unwrap();

let mut hashed_password = api::hash_password(&data_search.password);
let mut uri_acc = api::arg_to_api_route(&api::CheckableChoices::PASS, &hashed_password);
let uri_acc = api::arg_to_api_route(&api::CheckableChoices::PASS, &hashed_password);

set_checkpwn_panic!(api::errors::NETWORK_ERROR);
let mut pass_stat = client.get(&uri_acc).send().unwrap();
Expand All @@ -88,7 +88,6 @@ fn pass_check(data_search: &api::PassArg) {
}

// Zero out as this contains a weakly hashed password
uri_acc.zeroize();
hashed_password.zeroize();
}

Expand Down

0 comments on commit 7e0509c

Please sign in to comment.