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

[PATCH v5 0/5] config: add support for http.<url>.* settings

From
Kyle J. McKay <mackyle@gmail.com>
Date
Jul 15, 2013, 09:50 UTC
Message-ID
<506e5f642a838b95e0dd5b1f0fa1cfe@f74d39fa044aa309eaea14b9f57fe79>
This patch series adds support for url-specific http.* settings.

It has been suggested that a preparatory patch to address the way the http.sslCertPasswordProtected variable is handled be included.

I'm not sure if the intent was to make that a separate patch or include it in a patch series together with the http.<url>.* settings updates. I have, however, included it here this time along with a second patch that does the same thing for the GIT_SSL_CERT_PASSWORD_PROTECTED environment variable.

There is talk about possibly altering GIT_SSL_NO_VERIFY, GIT_CURL_FTP_NO_EPSV and GIT_CURL_VERBOSE to behave similarly. No patch is included here for that.

The http.<url>.* settings support addresses various feedback. To better support matching URLs that are equivalent but spelled differently, a url_normalize function has been added. Currently this patch leaves it in http.c as http_options_url_normalize as I am unclear whether it should go into url.{h,c} at this time since only http.c uses.

Since the url_normalize function's behavior is non-trivial, it is presented as a separate patch on top of the basic http.<url>.* settings support. A new test for it has also been included as the final patch. I am unclear on the proper number for this test, but have gone ahead and put it with the other http tests since this patch series places the url_normalize function into http.c.

Junio C Hamano (1):
  http.c: fix parsing of http.sslCertPasswordProtected variable
Kyle J. McKay (4):
  http.c: fix parsing of GIT_SSL_CERT_PASSWORD_PROTECTED variable
  config: add support for http.<url>.* settings
  config: improve support for http.<url>.* settings
  tests: add new test for the url_normalize function
 .gitignore               |   1 +
 Documentation/config.txt |  14 ++
 Makefile                 |   5 +
 http.c                   | 397 ++++++++++++++++++++++++++++++++++++++++++++---
 t/t5200-url-normalize.sh | 109 +++++++++++++
 test-url-normalize.c     |  62 ++++++++
 6 files changed, 565 insertions(+), 23 deletions(-)
 create mode 100755 t/t5200-url-normalize.sh
 create mode 100644 test-url-normalize.c
-- 
1.8.3
Next: Kyle J. McKay
Message 1 of 9 in “config: add support for http.<url>.* settings”
  1. 0/5 config: add support for http.<url>.* settingsKyle J. McKay, Jul 15, 2013
  2. 1/5 http.c: fix parsing of http.sslCertPasswordProtected variableKyle J. McKay, Jul 15, 2013
  3. 2/5 http.c: fix parsing of GIT_SSL_CERT_PASSWORD_PROTECTED variableKyle J. McKay, Jul 15, 2013
  4. 3/5 config: add support for http.<url>.* settingsKyle J. McKay, Jul 15, 2013
  5. 4/5 config: improve support for http.<url>.* settingsKyle J. McKay, Jul 15, 2013
  6. Eric SunshineJul 15, 2013
  7. Kyle J. McKayJul 16, 2013
  8. Eric SunshineJul 16, 2013
  9. 5/5 tests: add new test for the url_normalize functionKyle J. McKay, Jul 15, 2013

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.