couchdb-nano icon indicating copy to clipboard operation
couchdb-nano copied to clipboard

New Axios-based cookie support has side-effects because of axios-cookiejar-support

Open dsebastien opened this issue 4 years ago • 2 comments

I'm cross-posting this issue here just to let you know that the new usage of axios-cookiejar-support has side effects on unrelated code. As I've described here (https://github.com/auth0/node-jwks-rsa/issues/215), I'm using node-jwks-rsa, which also uses Axios as client. Unfortunately, since nano includes axios-cookiejar-support, it seems that the way node-jwks-rsa uses Axios does not work anymore.

I imagine that they could change the way they use Axios, but it's rather surprising to see such side-effects. Note that it's not an issue with nano per se, but I thought it might be interesting to make it appear here too, in case others stumble upon such problems.

Expected Behavior

No side-effects with Axios.

Current Behavior

Axios breaks when used in certain ways because of the presence of axios-cookiejar-support.

Possible Solution

Fix of axios-cookiejar-support & upgrade of nano (maybe it should also be a peer dependency?)

Steps to Reproduce (for bugs)

  • Add nano to a project
  • Import Axios using import {get} from 'axios' and try to make a simple request

Your Environment

  • Version used: 9.0.3
  • Browser Name and version: N/A
  • Operating System and version (desktop or mobile): N/A
  • Link to your project: N/A

dsebastien avatar Feb 10 '21 11:02 dsebastien

Same problem. Please don't monkey patch dependencies!

mlegenhausen avatar Feb 15 '21 15:02 mlegenhausen

Same problem here, an upstream dependency fabric-common that uses nano, ends up breaking some of our nestjs downstream deps that require httpService (which uses axios)

gallak87 avatar Aug 26 '21 16:08 gallak87