Correctly parse response bodies as JSON where the Content-Type is application/scim+json

src/fetch-wrapper.ts

@@ -155,7 +155,7 @@ async function getResponseData(response: Response): Promise<any> {
 
   const mimetype = safeParse(contentType);
 
-  if (mimetype.type === "application/json") {
+  if (isJSONResponse(mimetype)) {
     let text = "";
     try {
       text = await response.text();
@@ -173,6 +173,10 @@ async function getResponseData(response: Response): Promise<any> {
   }
 }
 
+function isJSONResponse(mimetype: { type: string }) {
+  return mimetype.type === "application/json" || mimetype.type === "application/scim+json";
+}
+
 function toErrorMessage(data: string | ArrayBuffer | Record<string, unknown>) {
   if (typeof data === "string") {
     return data;

test/request.test.ts

@@ -869,7 +869,7 @@ x//0u+zd/R/QRUzLOw4N72/Hu+UG6MNt5iDZFCtapRaKt6OvSBwy8w==
         required_pull_request_reviews: {
           required_approving_review_count: 1,
           dismiss_stale_reviews: true,
-          require_code_owner_reviews: true,
+          require_code owner_reviews: true,
           dismissal_restrictions: { users: [], teams: [] },
         },
         restrictions: { users: [], teams: [] },
@@ -1263,4 +1263,41 @@ x//0u+zd/R/QRUzLOw4N72/Hu+UG6MNt5iDZFCtapRaKt6OvSBwy8w==
       }),
     ).rejects.toHaveProperty("message", "Unknown error: {}");
   });
+
+  it("parses response bodies as JSON when Content-Type is application/scim+json", async () => {
+    expect.assertions(1);
+
+    const mock = fetchMock.createInstance();
+    mock.get("https://api.github.com/scim/v2/Users", {
+      status: 200,
+      body: {
+        totalResults: 1,
+        Resources: [
+          {
+            id: "123",
+            userName: "octocat",
+          },
+        ],
+      },
+      headers: {
+        "Content-Type": "application/scim+json",
+      },
+    });
+
+    const response = await request("GET /scim/v2/Users", {
+      request: {
+        fetch: mock.fetchHandler,
+      },
+    });
+
+    expect(response.data).toEqual({
+      totalResults: 1,
+      Resources: [
+        {
+          id: "123",
+          userName: "octocat",
+        },
+      ],
+    });
+  });
 });