Skip to content

Commit a4a4439

Browse files
jszakmeistergitster
authored andcommitted
http: drop the check for an empty proxy password before approving
credential_approve() already checks for a non-empty password before saving, so there's no need to do the extra check here. Signed-off-by: John Szakmeister <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent cd27f60 commit a4a4439

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

http.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1635,8 +1635,7 @@ static int handle_curl_result(struct slot_results *results)
16351635

16361636
if (results->curl_result == CURLE_OK) {
16371637
credential_approve(&http_auth);
1638-
if (proxy_auth.password)
1639-
credential_approve(&proxy_auth);
1638+
credential_approve(&proxy_auth);
16401639
credential_approve(&cert_auth);
16411640
return HTTP_OK;
16421641
} else if (results->curl_result == CURLE_SSL_CERTPROBLEM) {

0 commit comments

Comments
 (0)