Bug: Windows upload helper should quote URL-derived remote paths
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 78/100
Rechercherichtung
Beginne mit windows-release/merge-and-upload.py, insbesondere mit url2path, call_ssh, upload_ssh und der Upload-Schleife, die Verzeichnisse vorbereitet und Modi aktualisiert. Führe den mitgelieferten Reproducer aus und überprüfe jeden aufgezeichneten SSH- und SCP-Befehl. Erledigt ist die Aufgabe, wenn aus URLs abgeleitete Remote-Pfade in Verzeichnis-, Upload-, Gruppen- und Modusbefehlen als Pfaddaten behandelt werden und eine Regressionstestabdeckung für den Semikolonfall vorhanden ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
windows-release/merge-and-upload.py converts installer metadata URLs from
__install__.json into remote filesystem paths. The resulting paths are then
used in SSH and SCP command arguments for directory creation, upload, group, and
mode updates.
The metadata is normally produced by the release process, but the URL-derived
path should still be treated as path data and quoted before it is passed through
remote command strings.
Reproducer
Run this from the repository root on the current base tree. It mocks the
subprocess boundary and does not contact the upload host:
import importlib.util
from pathlib import Path
script = Path("windows-release/merge-and-upload.py")
spec = importlib.util.spec_from_file_location("merge_and_upload_repro", script)
module = importlib.util.module_from_spec(spec)
try:
spec.loader.exec_module(module)
except SystemExit:
# The script exits when no local __install__.json files are present. The
# upload helper functions are already defined by that point.
pass
calls = []
def fake_run(*args, single_cmd=False):
calls.append(args)
return ""
module._run = fake_run
module.PLINK = "plink.exe"
module.PSCP = "pscp.exe"
module.UPLOAD_HOST = "downloads.example.org"
module.UPLOAD_USER = "release-manager"
module.NO_UPLOAD = False
module.LOCAL_INDEX = False
dest = module.url2path(
"https://www.python.org/ftp/python/3.14.0;touch marker/"
"python-3.14.0-amd64.exe"
)
# This matches the directory preparation code in the base upload loop.
destdir = dest.rpartition("/")[0]
module.call_ssh(f"mkdir {destdir} && chgrp downloads {destdir} && chmod a+rx {destdir}")
module.upload_ssh("python-3.14.0-amd64.exe", dest)
for call in calls:
print(call)
The captured commands include the URL-derived path without shell quoting:
('plink.exe', '-batch', 'release-manager@downloads.example.org', 'mkdir /srv/www.python.org/ftp/python/3.14.0;touch marker && chgrp downloads /srv/www.python.org/ftp/python/3.14.0;touch marker && chmod a+rx /srv/www.python.org/ftp/python/3.14.0;touch marker')
('pscp.exe', '-batch', 'python-3.14.0-amd64.exe', 'release-manager@downloads.example.org:/srv/www.python.org/ftp/python/3.14.0;touch marker/python-3.14.0-amd64.exe')
('plink.exe', '-batch', 'release-manager@downloads.example.org', 'chgrp downloads /srv/www.python.org/ftp/python/3.14.0;touch marker/python-3.14.0-amd64.exe && chmod g-x,o+r /srv/www.python.org/ftp/python/3.14.0;touch marker/python-3.14.0-amd64.exe')
The semicolon from the metadata URL remains shell syntax in the generated
directory, upload, group, and mode commands.
Expected behavior
Remote paths derived from installer metadata URLs should be quoted as path data
before they are used in SSH command strings or SCP remote path arguments.
- Vorherrschende Sprache
- Python
- Sterne
- 61
- Forks
- 48
- Ø Merge
- 1 Std. 22 Min.
- Gemergte PRs (30 T.)
- 4
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus python/release-tools
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
python/release-tools#434 ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 38/100
python/release-tools#417 ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 68/100
python/release-tools#397 ·
-
bug
python/release-tools#286 · 1 zugewiesene Person ·
-
enhancement
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 52/100
python/release-tools#285 ·
Alle Issues in python/release-tools
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
use-agent-os/agent-os#3314 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
BasedHardware/omi#15662 · 1 Kommentar ·
-
documentation help wanted
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 90/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
AiursoftWeb/AnduinOS-2#19 ·