matrix-org / matrix-org/python-canonicaljson
There are no tests to ensure that floats fail
Offen
Dieses Issue hat noch niemand übernommen.
bug
help wanted
- Vorherrschende Sprache
- Python
- Sterne
- 30
- Forks
- 15
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
As per spec:
>Numbers in the JSON must be integers in the range `[-(2**53)+1, (2**53)-1]`.
There isn't any code that checks for floats, either.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die Zahlenserialisierung und die vorhandene Testsuite im Repository zu lokalisieren, und verfolge anschließend, wie numerische Werte verarbeitet werden. Füge eine Regressionstestabdeckung für Gleitkommazahlen und den angegebenen Ganzzahlbereich hinzu und führe die vollständige Testsuite aus, um zu bestätigen, dass ungültige Zahlen fehlschlagen, ohne unterstütztes kanonisches JSON zu beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- backend
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 38/100