Merge pull request #118 from thorgate/fetch-content-type-text

Use text as the default content type for fetch backend

.github/workflows/run-tests.yml

@@ -10,7 +10,7 @@ jobs:
     if: github.event_name == 'pull_request' || (github.event_name == 'push' && github.ref == 'refs/heads/master')
     strategy:
       matrix:
-        node: [10, 12, 14]
+        node: [12, 14, 16]
     steps:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1

README.md

@@ -17,23 +17,23 @@ Using NPM
 ```sh
 npm i tg-resources
 
-# And resource backend
-npm i @tg-resources/superagent
-
-# Or
+# And add fetch backend
 npm i @tg-resources/fetch
+
+# Or use superagent backend
+npm i @tg-resources/superagent
 ```
 
 Or using Yarn
 
 ```sh
 yarn add tg-resources
 
-# And resource backend
-yarn add @tg-resources/superagent
-
-# Or
+# And fetch backend
 yarn add @tg-resources/fetch
+
+# Or use superagent backend
+yarn add @tg-resources/superagent
 ```
 
 ### Does it work on react native?
@@ -52,7 +52,7 @@ import 'abortcontroller-polyfill/dist/polyfill-patch-fetch'
 
 ```js
 import { Router } from "tg-resources"
-import { SuperAgentResource: Resource } from "@tg-resources/superagent"
+import { FetchResource: Resource } from "@tg-resources/fetch"
 
 const onLoad = result => console.log(result);
 const onError = result => console.error(result);

packages/tg-resources-fetch/src/resource.ts

@@ -137,13 +137,10 @@ function parseFetchResponse(
             text: response.status === 204 ? null : '{}',
         });
     }
-    if (!response.headers.has('content-type')) {
-        // istanbul ignore next: Only happens w/ custom server that does not set Content-Type
-        throw new Error('Content type is missing from request');
-    }
 
     // Get content string to use correct parser
-    const contentType: string = response.headers.get('content-type') as string;
+    const contentType: string = (response.headers.get('content-type') ||
+        'text/plain') as string;
 
     if (contentType.includes('application/json')) {
         return response.json().then((body: any) => ({