Skip to content

http-auth-early

Michael J Gruber git 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest6 patches 3 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6url: decode buffers that are not NUL-terminatedhttp.c url.c url.h+23 -310 replies
2/6improve httpd auth testst/lib-httpd.sh t/t5550-http-fetch.sh+53 -80 replies
3/6remote-curl: don't retry auth failures with dumb protocolremote-curl.c+1 -10 replies
4/6http: retry authentication failures for all http requestshttp.c+11 -60 replies
5/6http: use hostname in credential descriptionhttp.c t/t5550-http-fetch.sh+35 -100 replies
6/6http_init: accept separate URL parameterhttp-fetch.c http-push.c http.c +2 more+8 -160 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/6] http-auth-early

Here are the early parts of Jeff's http-auth-keyring series. It contains only parts which are not using the credential API (which is still under discussion), so that this can go in (and help users) and alleviates the pressure on the credential discussion:

Early bits with cleanups to http.c.
Cherry-picked bit for improved prompts ("Username for ..." etc.) Cherry-pickes bit for using configured pushurls.

I tried to pick/resolve in a way which should help rebasing Jeff's series on top of this.

Jeff King (5):
url: decode buffers that are not NUL-terminated improve httpd auth tests
remote-curl: don't retry auth failures with dumb protocol
http: retry authentication failures for all http requests
http_init: accept separate URL parameter

Michael J Gruber (1):
http: use hostname in credential description

 http-fetch.c          |    2 +-
 http-push.c           |   10 +-----
 http.c                |   91 +++++++++++++++++++++++++++---------------------
 http.h                |    2 +-
 remote-curl.c         |    4 +-
 t/lib-httpd.sh        |   10 +++--
 t/t5550-http-fetch.sh |   51 +++++++++++++++++++++++++--
 url.c                 |   26 ++++++++++----
 url.h                 |    1 +
 9 files changed, 128 insertions(+), 69 deletions(-)
Signature · 1 line
1.7.7.338.g0156b