Skip to content

Commit

Permalink
Update git2
Browse files Browse the repository at this point in the history
  • Loading branch information
ehuss committed Apr 2, 2023
1 parent 0e474cf commit 78970bd
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 16 deletions.
8 changes: 3 additions & 5 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@ curl-sys = "0.4.61"
env_logger = "0.10.0"
filetime = "0.2.9"
flate2 = { version = "1.0.3", default-features = false, features = ["zlib"] }
git2 = "0.16.0"
git2-curl = "0.17.0"
git2 = "0.17.0"
git2-curl = "0.18.0"
gix = { version = "0.39.0", default-features = false, features = ["blocking-http-transport-curl", "progress-tree"] }
gix-features-for-configuration-only = { version = "0.28.0", package = "gix-features", features = [ "parallel" ] }
glob = "0.3.0"
Expand All @@ -47,9 +47,7 @@ jobserver = "0.1.26"
lazy_static = "1.2.0"
lazycell = "1.2.0"
libc = "0.2"
# Temporarily pin libgit2-sys due to some issues with SSH not working on
# Windows.
libgit2-sys = "=0.14.1"
libgit2-sys = "0.15.0"
log = "0.4.6"
memchr = "2.1.3"
opener = "0.5"
Expand Down
2 changes: 1 addition & 1 deletion crates/cargo-test-support/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ cargo-util = { path = "../cargo-util" }
crates-io = { path = "../crates-io" }
filetime = "0.2"
flate2 = { version = "1.0", default-features = false, features = ["zlib"] }
git2 = "0.16.0"
git2 = "0.17.0"
glob = "0.3"
itertools = "0.10.0"
lazy_static = "1.0"
Expand Down
4 changes: 2 additions & 2 deletions src/cargo/sources/git/utils.rs
Original file line number Diff line number Diff line change
Expand Up @@ -567,7 +567,7 @@ where
}

// Whelp, we tried our best
Err(git2::Error::from_str("no authentication available"))
Err(git2::Error::from_str("no authentication methods succeeded"))
});

// Ok, so if it looks like we're going to be doing ssh authentication, we
Expand Down Expand Up @@ -612,7 +612,7 @@ where
return git2::Cred::ssh_key_from_agent(&s);
}
}
Err(git2::Error::from_str("no authentication available"))
Err(git2::Error::from_str("no authentication methods succeeded"))
});

// If we made two attempts then that means:
Expand Down
2 changes: 1 addition & 1 deletion tests/testsuite/https.rs
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ fn self_signed_should_fail() {
.build();
// I think the text here depends on the curl backend.
let err_msg = if cfg!(target_os = "macos") {
"untrusted connection error; class=Ssl (16); code=Certificate (-17)"
"unexpected return value from ssl handshake -9806; class=Ssl (16)"
} else if cfg!(unix) {
"the SSL certificate is invalid; class=Ssl (16); code=Certificate (-17)"
} else if cfg!(windows) {
Expand Down
9 changes: 2 additions & 7 deletions tests/testsuite/ssh.rs
Original file line number Diff line number Diff line change
Expand Up @@ -469,11 +469,6 @@ Caused by:
failed to authenticate when downloading repository
*";
let err = if cfg!(windows) {
"error authenticating: unable to connect to agent pipe; class=Ssh (23)"
} else {
"error authenticating: failed connecting with agent; class=Ssh (23)"
};
let expected = if cargo_uses_gitoxide() {
format!(
"{shared_stderr} attempted to find username/password via `credential.helper`, but maybe the found credentials were incorrect
Expand All @@ -496,7 +491,7 @@ Caused by:
https://doc.rust-lang.org/cargo/reference/config.html#netgit-fetch-with-cli
Caused by:
{err}
no authentication methods succeeded
"
)
};
Expand Down Expand Up @@ -546,7 +541,7 @@ Caused by:
https://doc.rust-lang.org/cargo/reference/config.html#netgit-fetch-with-cli
Caused by:
{err}
no authentication methods succeeded
"
)
};
Expand Down

0 comments on commit 78970bd

Please sign in to comment.