Fix SCIM startIndex parsing (#2105)

Author: rhysyngsun

scim/views.py

@@ -172,8 +172,9 @@ def post(self, request, *args, **kwargs):  # noqa: ARG002
             msg = "Invalid schema uri. Must be SearchRequest."
             raise exceptions.BadRequestError(msg)
 
-        start = body.get("startIndex", 1)
-        count = body.get("count", 50)
+        # cast to ints because scim-for-keycloak sends strings
+        start = int(body.get("startIndex", 1))
+        count = int(body.get("count", 50))
         sort_by = body.get("sortBy", "id")
         sort_order = body.get("sortOrder", "ascending")
         query = body.get("filter", None)

scim/views_test.py

@@ -471,10 +471,12 @@ def _sort(user):
                 {
                     "schemas": [djs_constants.SchemaURI.SERACH_REQUEST],
                     "filter": " OR ".join([f'email EQ "{email}"' for email in emails]),
-                    "startIndex": start_index + 1,  # SCIM API is 1-based index
+                    # SCIM API is 1-based index
+                    # Additionally, scim-for-keycloak sends this as a string, but spec examples have ints
+                    "startIndex": str(start_index + 1),
                     **({"sortBy": sort_by} if sort_by is not None else {}),
                     **({"sortOrder": sort_order} if sort_order is not None else {}),
-                    **({"count": count} if count is not None else {}),
+                    **({"count": str(count)} if count is not None else {}),
                 }
             ),
         )