JSON deserialization does not handle OverflowError

Offen

#480 erstellt am 13.5.2025 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 52/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: python
Bereich: backend-api-design

Rechercherichtung

Beginne in packages/serialization/json/kiota_serialization_json/json_parse_node.py bei JsonParseNode.try_get_anything() und dem catch-Block um parse_timedelta_string(). Reproduziere die Eingabe "50001140846:00021" und prüfe die anderen im Issue erwähnten catch-Handler. Erledigt ist es, wenn fehlerhafte Werte unverändert durchgereicht werden, ohne einen nicht abgefangenen OverflowError auszulösen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Trying to deserialize certain misformatted strings result in an uncaught OverflowError exception instead of passing through the string unchanged. In particular, trying to deserialize a timedelta string such as "50001140846:00021" results in the following:

Traceback (most recent call last):
  File "lib\site-packages\kiota_abstractions\date_utils.py", line 66, in parse_timedelta_string
    return parse_timedelta_from_iso_format(text)
  File "lib\site-packages\kiota_abstractions\date_utils.py", line 34, in parse_timedelta_from_iso_format
    raise ValueError(f"Invalid ISO8601 duration string: {text}")
ValueError: Invalid ISO8601 duration string: 50001140846:00021

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
[...]
  File "lib\site-packages\kiota_serialization_json\json_parse_node.py", line 335, in try_get_anything
    return parse_timedelta_string(value)
  File "lib\site-packages\kiota_abstractions\date_utils.py", line 76, in parse_timedelta_string
    return timedelta(hours=hours, minutes=minutes, seconds=seconds)
OverflowError: days=2083380868; must have magnitude <= 999999999

The issue is that JsonParseNode.try_get_anything() only handles ValueError exceptions instead of also handling OverflowError. Changing this line to catch both ValueError and OverflowError allows deserialization to be successful:
https://github.com/microsoft/kiota-python/blob/b23edcbf268efdb81d14ef59cecc6c9565288248/packages/serialization/json/kiota_serialization_json/json_parse_node.py#L336

Most likely the other catch handlers in try_get_anything() should be updated as well.

Vorherrschende Sprache: Python
Sterne: 31
Forks: 39
Ø Merge: 13 Std. 21 Min.
Gemergte PRs (30 T.): 46

Beitragsleitfaden

Beitragsleitfaden öffnen

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 microsoft/kiota-python

ensure 304 response code does not throw Offen

help wanted type:bug

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

microsoft/kiota-python#477 · 2 Kommentare · Erstellt am 27.3.2025
Regression in microsoft-kiota-http 1.13.0: DriveItem content.get() returns None Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100

microsoft/kiota-python#745 · Erstellt am 21.9.2026
[Bug] Redirects not being followed Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100

microsoft/kiota-python#744 · 1 Kommentar · Erstellt am 21.9.2026
batch response bodies are dropped when they are JSON objects Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100

microsoft/kiota-python#737 · 1 Kommentar · Erstellt am 15.9.2026
Complete library typing and enforce mypy strict in CI Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100

microsoft/kiota-python#722 · 1 Kommentar · Erstellt am 05.9.2026

Alle Issues in microsoft/kiota-python