Skip to content

Commit 47814b7

Browse files
committed
fix: don't include schemas in GET query params
1 parent 20f6723 commit 47814b7

File tree

3 files changed

+6
-1
lines changed

3 files changed

+6
-1
lines changed

doc/changelog.rst

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,10 @@ Removed
1313
^^^^^^^
1414
- Support for Python 3.9.
1515

16+
Fixed
17+
^^^^^
18+
- ``schemas`` is no longer included in GET query parameters per RFC 7644 §3.4.2.
19+
1620
[0.6.1] - 2025-08-01
1721
--------------------
1822

scim2_client/client.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -424,6 +424,7 @@ def _prepare_query_request(
424424
elif isinstance(search_request, SearchRequest):
425425
payload = search_request.model_dump(
426426
exclude_unset=True,
427+
exclude={"schemas"},
427428
scim_ctx=Context.RESOURCE_QUERY_REQUEST,
428429
)
429430

tests/test_query.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -476,7 +476,7 @@ def test_response_bad_content_type(sync_client):
476476

477477

478478
def test_search_request(httpserver, sync_client):
479-
query_string = "schemas=urn%3Aietf%3Aparams%3Ascim%3Aapi%3Amessages%3A2.0%3ASearchRequest&attributes=userName&attributes=displayName&filter=userName+Eq+%22john%22&sortBy=userName&sortOrder=ascending&startIndex=1&count=10"
479+
query_string = "attributes=userName&attributes=displayName&filter=userName+Eq+%22john%22&sortBy=userName&sortOrder=ascending&startIndex=1&count=10"
480480

481481
httpserver.expect_request(
482482
"/Users/with-qs", query_string=query_string

0 commit comments

Comments
 (0)