BatchRequestItem keeps the absolute URL, and set_url() only matches //v1.0/

Offen Anfängerfreundlich

#1,116 erstellt am 03.9.2026 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: 88/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Aktiv
Tech-Stack: python
Bereich: api

Rechercherichtung

Beginne in msgraph_core/requests/batch_request_item.py und konzentriere dich auf API_VERSION_REGEX, init und set_url(). Reproduziere das Verhalten mit dem verlinkten 1.5.1 gist und überprüfe, dass ein BatchRequestItem im serialisierten Batch-Body eine relative URL ohne das Präfix /v1.0 oder /beta erzeugt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

BatchRequestItem built from a RequestInformation keeps the full https://graph.microsoft.com/v1.0/... URL, so the serialized batch body sends absolute URLs. set_url() is meant to strip the version prefix but its regex can't match a normal URL:

msgraph_core/requests/batch_request_item.py on main, same in 1.5.1 (and back to 1.3.8):

line 24: API_VERSION_REGEX = re.compile(r'/\/(v1.0|beta)/'), both / and \/ are literal slashes in Python, so it only matches //v1.0/. Looks like a JavaScript regex literal ported as-is.
line 50: __init__ keeps request_information.url after the me-token replace, never calls set_url.
lines 101-127: set_url() uses the regex above, so it never strips the version either.

Repro on 1.5.1: https://gist.github.com/HardMax71/156fbbad8a0ba78a789970a8ee550824

msgraph-core 1.5.1 kiota-abstractions 1.12.0
after __init__     url = 'https://graph.microsoft.com/v1.0/me'
after set_url()    url = '/v1.0/me'
API_VERSION_REGEX  = /\/(v1.0|beta)/
regex matches '/v1.0/me'?  False | matches '//v1.0/me'? True
serialized batch body: {"requests": [{"id": "m1", "method": "POST", "url": "https://graph.microsoft.com/v1.0/users/u1/messages/m1/move", ...}]}

Three-line fix: API_VERSION_REGEX = re.compile(r'/(v1\.0|beta)(?=/|$)'), and in __init__ replace the manual assignment with self.set_url(request_information.url) (it already handles the me-token, query string and fragment). We currently strip the prefix by hand before building items.

Vorherrschende Sprache: Python
Sterne: 288
Forks: 52
Ø Merge: 8 Std. 10 Min.
Gemergte PRs (30 T.): 1

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 microsoftgraph/msgraph-sdk-python-core

Logging to root logger Offen

status:waiting-for-triage type:bug

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

microsoftgraph/msgraph-sdk-python-core#1078 · Erstellt am 21.5.2026
Missing 'deprecated' dependency Offen

status:waiting-for-triage type:bug

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

microsoftgraph/msgraph-sdk-python-core#1030 · 2 Kommentare · 4 Reaktionen · Erstellt am 12.9.2025
Replace httpx with httpx2 Offen

status:waiting-for-triage type:feature

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

microsoftgraph/msgraph-sdk-python-core#1115 · Erstellt am 11.8.2026
BatchRequestItem loses query parameters from RequestInformation Offen

status:waiting-for-triage type:bug

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

microsoftgraph/msgraph-sdk-python-core#1035 · Erstellt am 02.12.2025
500 errors connecting to MS Graph Calendar API Offen

status:waiting-for-triage

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

microsoftgraph/msgraph-sdk-python-core#935 · 1 Reaktion · Erstellt am 31.5.2025

Alle Issues in microsoftgraph/msgraph-sdk-python-core