Nulo/forgejo
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix URL calculation in clone input box (#29470)

Browse source 
Ported the function as-is and added comments so we don't forget about
this in the future.

Fixes: https://github.com/go-gitea/gitea/issues/29462
(cherry picked from commit 82405f808d7b50c3580f26e5ca645e2ed6d284ab)
This commit is contained in:
silverwind 2024-02-28 16:04:04 +01:00 committed by Earl Warren
parent b0cadeb022
commit 6e89eff490
No known key found for this signature in database
GPG key ID: 0579CB2928A78A00
2 changed files with 18 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
templates/repo/clone_script.tmpl
View file

@ -24,14 +24,22 @@
const btn = isSSH ? sshBtn : httpsBtn; const btn = isSSH ? sshBtn : httpsBtn;
if (!btn) return; if (!btn) return;
let link = btn.getAttribute('data-link'); // NOTE: Keep this function in sync with the one in the js folder
if (link.startsWith('http://') || link.startsWith('https://')) { function toOriginUrl(urlStr) {
// use current protocol/host as the clone link try {
const url = new URL(link); if (urlStr.startsWith('http://') || urlStr.startsWith('https://') || urlStr.startsWith('/')) {
url.protocol = window.location.protocol; const {origin, protocol, hostname, port} = window.location;
url.host = window.location.host; const url = new URL(urlStr, origin);
link = url.toString(); url.protocol = protocol;
url.hostname = hostname;
url.port = port || (protocol === 'https:' ? '443' : '80');
return url.toString();
} }
} catch {}
return urlStr;
}
const link = toOriginUrl(btn.getAttribute('data-link'));
for (const el of document.getElementsByClassName('js-clone-url')) { for (const el of document.getElementsByClassName('js-clone-url')) {
el[el.nodeName === 'INPUT' ? 'value' : 'textContent'] = link; el[el.nodeName === 'INPUT' ? 'value' : 'textContent'] = link;
} }

5
web_src/js/webcomponents/GiteaOriginUrl.js
View file

@ -1,7 +1,8 @@
// Convert an absolute or relative URL to an absolute URL with the current origin // Convert an absolute or relative URL to an absolute URL with the current origin. It only
// processes absolute HTTP/HTTPS URLs or relative URLs like '/xxx' or '//host/xxx'.
// NOTE: Keep this function in sync with clone_script.tmpl
export function toOriginUrl(urlStr) { export function toOriginUrl(urlStr) {
try { try {
// only process absolute HTTP/HTTPS URL or relative URLs ('/xxx' or '//host/xxx')
if (urlStr.startsWith('http://') || urlStr.startsWith('https://') || urlStr.startsWith('/')) { if (urlStr.startsWith('http://') || urlStr.startsWith('https://') || urlStr.startsWith('/')) {
const {origin, protocol, hostname, port} = window.location; const {origin, protocol, hostname, port} = window.location;
const url = new URL(urlStr, origin); const url = new URL(urlStr, origin);