metadata API: serialization is missing a final newline
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 38/100
Rechercherichtung
Beginne bei JSONSerializer.serialize(), untersuche anschließend Metadata.to_file() und Metadata.to_bytes(), um ihre aktuelle Ausgabe zu vergleichen. Bestätige das erwartete Verhalten bezüglich des abschließenden Zeilenumbruchs und bewerte die Auswirkungen auf die Kompatibilität der Serialisierungs-API, bevor du entscheidest, welche Tests oder Dokumentation geändert werden müssen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Our JSON serialization is basically json.dumps(metadata_obj.to_dict(), ...).encode("utf-8")
json.dumps() return value does not end in a newline. This means any files written with this content will not end in a newline: this makes the produced files not valid posix text files.
For Metadata.to_file() this seems like a minor bug (and Metadata.to_bytes() should have the same output for consistency), so in practice JSONSerializer.serialize() probably should add the final newline.
My only concern about this is that it is an unexpected change in the serialization output... IF we change this, we should definitely consider this an API change for users who produce metadata.
- Vorherrschende Sprache
- Python
- Sterne
- 1.7k
- Forks
- 304
- Ø Merge
- 9 Std. 25 Min.
- Gemergte PRs (30 T.)
- 14
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
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 theupdateframework/python-tuf
-
switch to main branch?Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
theupdateframework/python-tuf#3001 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 42/100
theupdateframework/python-tuf#2979 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Use Immutable ReleasesOffenenhancement github_actions
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
theupdateframework/python-tuf#2920 · 1 Kommentar · 2 Reaktionen ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
theupdateframework/python-tuf#2842 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
theupdateframework/python-tuf#2836 · 7 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in theupdateframework/python-tuf
Ähnliche Issues
-
customer-reported
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Azure/azure-cli#34150 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
community-request
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 95/100
NVIDIA-NeMo/Curator#2464 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
weblate-discover crashes with an unhandled FileNotFoundError when the directory does not existOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
WeblateOrg/translation-finder#1099 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
trezor/trezor-firmware#7997 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
Maintainer antworten meist innerhalb von 1 Tag