fix: vendored json content types being treated as plain text

readmeio · Aug 3, 2020 · 5b70186 · 5b70186
1 parent de44871
commit 5b70186
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/packages/api-explorer/__tests__/lib/content-type-is-json.test.js b/packages/api-explorer/__tests__/lib/content-type-is-json.test.js
@@ -11,6 +11,11 @@ describe('contentTypeIsJson', () => {
     expect(contentTypeIsJson(contentType)).toBe(true);
   });
 
+  it('should return true if application/vnd.github.v3.star+json', () => {
+    const contentType = 'application/vnd.github.v3.star+json';
+    expect(contentTypeIsJson(contentType)).toBe(true);
+  });
+
   it('should return false otherwise', () => {
     const contentType = 'text/html';
     expect(contentTypeIsJson(contentType)).toBe(false);

diff --git a/packages/api-explorer/src/lib/content-type-is-json.js b/packages/api-explorer/src/lib/content-type-is-json.js
@@ -1,5 +1,5 @@
 function contentTypeIsJson(contentType) {
-  const jsonContentTypes = ['application/json', 'application/vnd.api+json'];
+  const jsonContentTypes = ['application/json', '+json'];
   return jsonContentTypes.some(ct => contentType.includes(ct));
 }