* [PATCH 1/3] urlmatch: normalize ssh and ftp default ports
2026-07-24 0:40 [PATCH 0/3] config: support scp-style --url Fabian Pottbäcker
@ 2026-07-24 0:40 ` Fabian Pottbäcker
2026-07-24 0:40 ` [PATCH 2/3] t1300: cover --url for some --get-urlmatch tests Fabian Pottbäcker
2026-07-24 0:40 ` [PATCH 3/3] config: use url_parse for --url Fabian Pottbäcker
2 siblings, 0 replies; 4+ messages in thread
From: Fabian Pottbäcker @ 2026-07-24 0:40 UTC (permalink / raw)
To: git; +Cc: Junio C Hamano, Patrick Steinhardt, Fabian Pottbäcker
These protocols are still supported by git and have well known
default ports. This leaves FTPS, which does not have one default
port.
Signed-off-by: Fabian Pottbäcker <fpottbaecker+git@mailbox.org>
---
t/unit-tests/u-urlmatch-normalization.c | 9 +++++++++
urlmatch.c | 16 ++++++++++++----
2 files changed, 21 insertions(+), 4 deletions(-)
diff --git a/t/unit-tests/u-urlmatch-normalization.c b/t/unit-tests/u-urlmatch-normalization.c
index 3595d893a2..1808e5e51f 100644
--- a/t/unit-tests/u-urlmatch-normalization.c
+++ b/t/unit-tests/u-urlmatch-normalization.c
@@ -141,9 +141,18 @@ void test_urlmatch_normalization__port_normalization(void)
check_normalized_url("http://x:80", "http://x/");
check_normalized_url("http://x:080", "http://x/");
check_normalized_url("http://x:000000080", "http://x/");
+ check_normalized_url("https://x:8443", "https://x:8443/");
check_normalized_url("https://x:443", "https://x/");
check_normalized_url("https://x:0443", "https://x/");
check_normalized_url("https://x:000000443", "https://x/");
+ check_normalized_url("ftp://x:2121", "ftp://x:2121/");
+ check_normalized_url("ftp://x:21", "ftp://x/");
+ check_normalized_url("ftp://x:021", "ftp://x/");
+ check_normalized_url("ftp://x:00000021", "ftp://x/");
+ check_normalized_url("ssh://x:2222", "ssh://x:2222/");
+ check_normalized_url("ssh://x:22", "ssh://x/");
+ check_normalized_url("ssh://x:022", "ssh://x/");
+ check_normalized_url("ssh://x:00000022", "ssh://x/");
}
void test_urlmatch_normalization__general_escape(void)
diff --git a/urlmatch.c b/urlmatch.c
index 20bc2d009c..0c2ddf2e40 100644
--- a/urlmatch.c
+++ b/urlmatch.c
@@ -274,12 +274,20 @@ static char *url_normalize_1(const char *url, struct url_info *out_info, bool al
if (url == slash_ptr) {
/* Skip ":" port with no number, it's same as default */
} else if (slash_ptr - url == 2 &&
- starts_with(norm.buf, "http:") &&
- !strncmp(url, "80", 2)) {
+ starts_with(norm.buf, "ftp:") &&
+ !strncmp(url, "21", 2)) {
+ /* Skip http :21 as it's the default */
+ } else if (slash_ptr - url == 2 &&
+ starts_with(norm.buf, "ssh:") &&
+ !strncmp(url, "22", 2)) {
+ /* Skip http :22 as it's the default */
+ } else if (slash_ptr - url == 2 &&
+ starts_with(norm.buf, "http:") &&
+ !strncmp(url, "80", 2)) {
/* Skip http :80 as it's the default */
} else if (slash_ptr - url == 3 &&
- starts_with(norm.buf, "https:") &&
- !strncmp(url, "443", 3)) {
+ starts_with(norm.buf, "https:") &&
+ !strncmp(url, "443", 3)) {
/* Skip https :443 as it's the default */
} else {
/*
--
2.50.1 (Apple Git-155)
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH 3/3] config: use url_parse for --url
2026-07-24 0:40 [PATCH 0/3] config: support scp-style --url Fabian Pottbäcker
2026-07-24 0:40 ` [PATCH 1/3] urlmatch: normalize ssh and ftp default ports Fabian Pottbäcker
2026-07-24 0:40 ` [PATCH 2/3] t1300: cover --url for some --get-urlmatch tests Fabian Pottbäcker
@ 2026-07-24 0:40 ` Fabian Pottbäcker
2 siblings, 0 replies; 4+ messages in thread
From: Fabian Pottbäcker @ 2026-07-24 0:40 UTC (permalink / raw)
To: git; +Cc: Junio C Hamano, Patrick Steinhardt, Fabian Pottbäcker
This enables the use of the scp-style URL syntax for this option and
the depracated --get-urlmatch. Since URL matching was primarily used
for http(s) transports previously (with the potential exception of
promisor.acceptFromServerUrl), this has little internal effect and
mostly brings the behaviour of this URL argument in line with other
commands.
Signed-off-by: Fabian Pottbäcker <fpottbaecker+git@mailbox.org>
---
Documentation/git-config.adoc | 5 ++++
builtin/config.c | 2 +-
t/t1300-config.sh | 44 +++++++++++++++++++++++++++++++++++
3 files changed, 50 insertions(+), 1 deletion(-)
diff --git a/Documentation/git-config.adoc b/Documentation/git-config.adoc
index 57af010ade..a38d68512e 100644
--- a/Documentation/git-config.adoc
+++ b/Documentation/git-config.adoc
@@ -143,6 +143,11 @@ permitted).
<section>.<key> is used as a fallback). When given just the
<section> as name, do so for all the keys in the section and
list them. Returns error code 1 if no value is found.
++
+This option supports all URL formats (see linkgit:git-fetch[1]),
+unlike the config file URL syntax explained in the `http.<url>.*`
+options, which requires an explicitly specified scheme and does
+not support the scp-style URL syntax.
--global::
For writing options: write to global `~/.gitconfig` file
diff --git a/builtin/config.c b/builtin/config.c
index 8d8ec0beea..b5488a9497 100644
--- a/builtin/config.c
+++ b/builtin/config.c
@@ -870,7 +870,7 @@ static int get_urlmatch(const struct config_location_options *opts,
config.cascade_fn = NULL;
config.cb = &values;
- if (!url_normalize(url, &config.url))
+ if (!url_parse(url, &config.url))
die("%s", config.url.err);
config.section = section = xstrdup_tolower(var);
diff --git a/t/t1300-config.sh b/t/t1300-config.sh
index 8310fe6a65..477e030cf3 100755
--- a/t/t1300-config.sh
+++ b/t/t1300-config.sh
@@ -1907,6 +1907,50 @@ test_expect_success 'urlmatch' '
test_cmp expect actual
'
+test_expect_success 'urlmatch with scp syntax' '
+ cat >.git/config <<-\EOF &&
+ [section]
+ flag = false
+ [section "ssh://example.com"]
+ flag = true
+ key = value
+ EOF
+
+ echo "fatal: invalid URL scheme name or missing '"'"'://'"'"' suffix" >expect_err &&
+ test_expect_code 128 git config --bool --get-urlmatch section.flag example.com >actual 2>error &&
+ test_cmp expect_err error &&
+ test_must_be_empty actual &&
+ test_expect_code 128 git config get --url=example.com --bool section.flag >actual 2>error &&
+ test_cmp expect_err error &&
+ test_must_be_empty actual &&
+
+ test_expect_code 1 git config --bool --get-urlmatch doesnt.exist example.com: >actual &&
+ test_must_be_empty actual &&
+ test_expect_code 1 git config get --url=example.com: --bool doesnt.exist >actual &&
+ test_must_be_empty actual &&
+
+ echo true >expect &&
+ git config --bool --get-urlmatch section.flag git@example.com:path >actual &&
+ test_cmp expect actual &&
+ git config get --bool --url=git@example.com:path section.flag >actual &&
+ test_cmp expect actual &&
+
+ echo false >expect &&
+ git config --bool --get-urlmatch section.flag https://example.com >actual &&
+ test_cmp expect actual &&
+ git config get --bool --url=https://example.com section.flag >actual &&
+ test_cmp expect actual &&
+
+ {
+ echo section.flag true &&
+ echo section.key value
+ } >expect &&
+ git config --get-urlmatch section git@example.com:path >actual &&
+ test_cmp expect actual &&
+ git config get --url=git@example.com:path section >actual &&
+ test_cmp expect actual
+'
+
test_expect_success 'urlmatch with --show-scope' '
cat >.git/config <<-\EOF &&
[http "https://weak.example.com"]
--
2.50.1 (Apple Git-155)
^ permalink raw reply related [flat|nested] 4+ messages in thread