git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2 1/2] http: store credential when PKI auth is used

From
Jeff King <peff@peff.net>
Date
Mar 12, 2021, 01:45 UTC
Message-ID
<YErHsQwIC2grgjwI@coredump.intra.peff.net>
In-Reply-To
<YErGymyECXjPXWcP@camp.crustytoothpaste.net>
On Fri, Mar 12, 2021 at 01:41:30AM +0000, brian m. carlson wrote:
Show 21 quoted lines
> > diff --git a/http.c b/http.c
> > index f8ea28bb2e..12a8aaba48 100644
> > --- a/http.c
> > +++ b/http.c
> > @@ -1637,7 +1637,17 @@ static int handle_curl_result(struct slot_results *results)
> >  		credential_approve(&http_auth);
> >  		if (proxy_auth.password)
> >  			credential_approve(&proxy_auth);
> > +		credential_approve(&cert_auth);
> >  		return HTTP_OK;
> > +	} else if (results->curl_result == CURLE_SSL_CERTPROBLEM) {
> > +		/*
> > +		 * We can't tell from here whether it's a bad path, bad
> > +		 * certificate, bad password, or something else wrong
> > +		 * with the certificate.  So we reject the credential to
> > +		 * avoid caching or saving a bad password.
> > +		 */
> > +		credential_reject(&http_auth);
> 
> Is this supposed to be &cert_auth here?  I'm not sure how a bad HTTP
> password would even have been tested in this case.

Good catch! When reviewing, I was so busy thinking about _where_ this line should go that I didn't even notice what it said. :)

-Peff
Previous: brian m. carlsonNext: John Szakmeister
Message 5 of 9 in “http: store credential when PKI auth is used”
  1. 0/2 http: store credential when PKI auth is usedJohn Szakmeister, Mar 12, 2021
  2. 1/2 http: store credential when PKI auth is usedJohn Szakmeister, Mar 12, 2021
  3. Junio C HamanoMar 12, 2021
  4. brian m. carlsonMar 12, 2021
  5. Jeff KingMar 12, 2021
  6. John SzakmeisterMar 12, 2021
  7. 2/2 http: drop the check for an empty proxy password before approvingJohn Szakmeister, Mar 12, 2021
  8. Jeff KingMar 12, 2021
  9. John SzakmeisterMar 12, 2021

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.