Re: [PATCH] http.c: prompt for username on 403

On 12/10/25 07:35, brian m. carlson wrote:

On 2025-12-09 at 08:22:49, Ashlesh Gawande wrote:

quoted

I was working on separating the tests and thought about the original
proposal a bit more.
To stop the credentials from being erased on 403 could something like the
following be acceptable?

         else if (results->http_code == 401 || results->http_code == 403) {
                 if ((http_auth.username && http_auth.password) ||\
                     (http_auth.authtype && http_auth.credential)) {
+                       // Do not override existing credentials on 403
+                       if (results->http_code == 403) {
+                               return HTTP_ERROR;
+                       }
+
                         if (http_auth.multistage) {

So then we would prompt on 403 only if credentials are not configured.

Can you tell me what file you see this in?  I don't actually see any
place in the code that has "http_code == 403" in the latest version of
the main branch.

I wonder if your issue may already be fixed in a newer version than you
have.

Oh, that http_code == 403 is my original proposal to prompt for 
username/password on 403 (I did the diff on top of that instead of base).
But you pointed out that it would wipe out existing credentials. This is 
an attempt to fix that by not prompting on 403 if git credentials are set.
So when credentials are provided through default netrc file (such that 
http_auth.* are not set; git credential helper is not set) then we can 
still get the prompt on 403.