From fa871b31f6c3309d3c3d0b4bebcd5b5b62706f0a Mon Sep 17 00:00:00 2001
From: Bryce Kwon <bryce@brycekwon.com>
Date: Sat, 29 Aug 2026 15:50:14 -1000
Subject: Point clone rows at a url a browser can follow

---
 source/ui-shared.c | 69 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 69 insertions(+)

(limited to 'source/ui-shared.c')

diff --git a/source/ui-shared.c b/source/ui-shared.c
index 85712bf..5cf9428 100644
--- a/source/ui-shared.c
+++ b/source/ui-shared.c
@@ -317,8 +317,26 @@ static void print_rel_date(time_t t, int tz, double count, const char *class,
 	htmlf("'>%.0f %s</time>", count, suffix);
 }
 
+/*
+ * Whether a clone url opens with a URI scheme. The scp form git@host:path
+ * has none, and a link element's href must hold a URI, so that form stays
+ * out of the document head.
+ */
+static int has_uri_scheme(const char *url)
+{
+	const char *p = url;
+
+	if (!isalpha((unsigned char)*p))
+		return 0;
+	while (isalnum((unsigned char)*p) || *p == '+' || *p == '-' || *p == '.')
+		p++;
+	return *p == ':';
+}
+
 static void print_rel_vcs_link(const char *url)
 {
+	if (!has_uri_scheme(url))
+		return;
 	html("<link rel='vcs-git' href='");
 	html_attr(url);
 	html("' title='");
@@ -1620,6 +1638,57 @@ void cgit_add_clone_urls(void (*fn)(const char *))
 		add_clone_urls(fn, ctx.cfg.clone_prefix, ctx.repo->url);
 }
 
+static char *http_clone_url;
+static int http_clone_url_scanned;
+
+// Scheme names compare case-insensitively.
+static int is_http_url(const char *url)
+{
+	return istarts_with(url, "http://") || istarts_with(url, "https://");
+}
+
+static void capture_http_clone_url(const char *url)
+{
+	if (!http_clone_url && is_http_url(url))
+		http_clone_url = xstrdup(url);
+}
+
+/*
+ * One row of the clone table. The url is shown as written, and every row is
+ * a link, but a browser cannot follow the ssh, scp or git forms, so those
+ * rows point at the first http url in the clone list, or at the repository's
+ * own page when the list has none, which with http clone enabled is itself a
+ * working clone url. A colspan of zero leaves the attribute out.
+ */
+void cgit_print_clone_row(const char *url, int colspan)
+{
+	const char *href = url;
+	char *fallback = NULL;
+
+	if (!is_http_url(url)) {
+		if (!http_clone_url_scanned) {
+			http_clone_url_scanned = 1;
+			cgit_add_clone_urls(capture_http_clone_url);
+		}
+		href = http_clone_url;
+		if (!href)
+			href = fallback = cgit_repourl(ctx.repo->url);
+	}
+
+	if (colspan)
+		htmlf("<tr><td colspan='%d' class='clone-url'>", colspan);
+	else
+		html("<tr><td class='clone-url'>");
+	html("<a rel='vcs-git' title='");
+	html_attr(ctx.repo->name);
+	html(" Git repository' href='");
+	html_attr(href);
+	html("'>");
+	html_txt(url);
+	html("</a></td></tr>\n");
+	free(fallback);
+}
+
 void cgit_add_hidden_formfields(int incl_head, int incl_search,
 				const char *page)
 {
-- 
cgit v2.8.0