Update axios version to 1.6.4 and fix Content-Type header issue

Author: dphuang2

README.md

@@ -2,6 +2,9 @@
 
 Works in:
 
- - Node >= 16
- - Browser
- - Next.js Edge Runtime
+- Node >= 16
+- Browser
+- Next.js Edge Runtime
+
+The version number of this package is the same as the version number of the `axios`
+version it is compatible with.

index.ts

@@ -1,8 +1,10 @@
 import {
   AxiosAdapter,
   AxiosError,
+  AxiosHeaders,
   AxiosRequestConfig,
   AxiosResponse,
+  InternalAxiosRequestConfig,
 } from "axios";
 import settle from "./settle";
 import buildURL from "./helpers/buildURL";
@@ -24,6 +26,16 @@ import { isUndefined, isStandardBrowserEnv, isFormData } from "./utils";
  * - Check if timeout
  */
 const fetchAdapter: AxiosAdapter = async (config) => {
+  if (config.headers) {
+    // In axios version >= 1.0.0, a falsy value ("undefined") for Content-Type
+    // is automatically set for some reason This can cause issues if the server
+    // expects Content-Type header to be something meaningful. To avoid this,
+    // we unset Content-Type if it's falsy (e.g. "undefined").
+    if (!config.headers.getContentType()) {
+      config.headers.clear("Content-Type");
+    }
+  }
+
   const request = createRequest(config);
   const promiseChain = [getResponse(request, config)];
 
@@ -56,7 +68,7 @@ const fetchAdapter: AxiosAdapter = async (config) => {
  */
 async function getResponse(
   request: Request,
-  config: AxiosRequestConfig
+  config: InternalAxiosRequestConfig
 ): Promise<AxiosResponse | Error> {
   let stageOne: Response;
   try {
@@ -116,7 +128,7 @@ async function getResponse(
     status: stageOne.status,
     statusText: stageOne.statusText,
     headers: Object.fromEntries(Object.entries(stageOne.headers)), // Make a copy of headers
-    config: config,
+    config: { ...config, headers: new AxiosHeaders(config.headers) },
     request,
   };
 
@@ -197,7 +209,7 @@ function createRequest(config: AxiosRequestConfig): Request {
  */
 function createError(
   message: string,
-  config: AxiosRequestConfig,
+  config: InternalAxiosRequestConfig,
   code: string,
   request: Request,
   response?: AxiosResponse

package.json

@@ -1,6 +1,6 @@
 {
   "name": "konfig-axios-fetch-adapter",
-  "version": "1.0.9",
+  "version": "1.6.4",
   "description": "Fetch adapter for axios written in TypeScript",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -33,7 +33,7 @@
   "devDependencies": {
     "@types/jest": "^29.5.3",
     "@types/node": "^18.15.11",
-    "axios": "^0.27.2",
+    "axios": "^1.6.4",
     "eventsource-parser": "^1.0.0",
     "jest": "^29.6.1",
     "konfig-release-it": "^15.5.1",
@@ -50,7 +50,7 @@
     }
   },
   "peerDependencies": {
-    "axios": ">=0.26.0"
+    "axios": ">=1.6.4"
   },
   "packageManager": "[email protected]",
   "dependencies": {

settle.ts

@@ -2,8 +2,6 @@ import { AxiosError } from "axios";
 
 export default function settle(resolve, reject, response) {
   var validateStatus = response.config.validateStatus;
-  console.log("test");
-  debugger;
   if (!response.status || !validateStatus || validateStatus(response.status)) {
     resolve(response);
   } else {

tsconfig.json

@@ -7,6 +7,7 @@
     "forceConsistentCasingInFileNames": true /* Ensure that casing is correct in imports. */,
     "skipLibCheck": true /* Skip type checking all .d.ts files. */,
     "declaration": true,
+    "sourceMap": true,
     "outDir": "./dist"
   },
   "include": ["index.ts"]

yarn.lock

@@ -1411,13 +1411,14 @@ __metadata:
   languageName: node
   linkType: hard
 
-"axios@npm:^0.27.2":
-  version: 0.27.2
-  resolution: "axios@npm:0.27.2"
+"axios@npm:^1.6.4":
+  version: 1.6.4
+  resolution: "axios@npm:1.6.4"
   dependencies:
-    follow-redirects: ^1.14.9
+    follow-redirects: ^1.15.4
     form-data: ^4.0.0
-  checksum: 38cb7540465fe8c4102850c4368053c21683af85c5fdf0ea619f9628abbcb59415d1e22ebc8a6390d2bbc9b58a9806c874f139767389c862ec9b772235f06854
+    proxy-from-env: ^1.1.0
+  checksum: 48d8af8488ac7402fae312437c0189b3b609a472fca2f7fc796129c804d98520589b6317096eba8509711d49f855a3f620b6a24ff23acd73ac26433d0383b8f9
   languageName: node
   linkType: hard
 
@@ -2804,13 +2805,13 @@ __metadata:
   languageName: node
   linkType: hard
 
-"follow-redirects@npm:^1.14.9":
-  version: 1.15.3
-  resolution: "follow-redirects@npm:1.15.3"
+"follow-redirects@npm:^1.15.4":
+  version: 1.15.4
+  resolution: "follow-redirects@npm:1.15.4"
   peerDependenciesMeta:
     debug:
       optional: true
-  checksum: 584da22ec5420c837bd096559ebfb8fe69d82512d5585004e36a3b4a6ef6d5905780e0c74508c7b72f907d1fa2b7bd339e613859e9c304d0dc96af2027fd0231
+  checksum: e178d1deff8b23d5d24ec3f7a94cde6e47d74d0dc649c35fc9857041267c12ec5d44650a0c5597ef83056ada9ea6ca0c30e7c4f97dbf07d035086be9e6a5b7b6
   languageName: node
   linkType: hard
 
@@ -4470,7 +4471,7 @@ __metadata:
   dependencies:
     "@types/jest": ^29.5.3
     "@types/node": ^18.15.11
-    axios: ^0.27.2
+    axios: ^1.6.4
     eventsource-parser: ^1.0.0
     jest: ^29.6.1
     konfig-release-it: ^15.5.1
@@ -4481,7 +4482,7 @@ __metadata:
     vite: ^2.8.0
     web-streams-polyfill: ^3.2.1
   peerDependencies:
-    axios: ">=0.26.0"
+    axios: ">=1.6.4"
   languageName: unknown
   linkType: soft
 
@@ -5459,7 +5460,7 @@ __metadata:
   languageName: node
   linkType: hard
 
-"proxy-from-env@npm:^1.0.0":
+"proxy-from-env@npm:^1.0.0, proxy-from-env@npm:^1.1.0":
   version: 1.1.0
   resolution: "proxy-from-env@npm:1.1.0"
   checksum: ed7fcc2ba0a33404958e34d95d18638249a68c430e30fcb6c478497d72739ba64ce9810a24f53a7d921d0c065e5b78e3822759800698167256b04659366ca4d4