node-soap icon indicating copy to clipboard operation
node-soap copied to clipboard

updated axios and axios-ntlm to latest

Open vision10 opened this issue 2 years ago • 33 comments
trafficstars

latest version of axios-ntlm fixes node v18 digital envelope error Error: error:0308010C:digital envelope routines::unsupported

updated axios to latest: 1.3.2 updated axios-ntlm to latest: 1.4.0 (uses axios 1.3.2)

also 2 tests where failing because of hardcoded response Content-Length

this request needs to be accepted to pass all tests: https://github.com/vpulim/node-soap/pull/1206

vision10 avatar Feb 10 '23 14:02 vision10

Would like to see this update as well!

h3llrais3r avatar Mar 09 '23 09:03 h3llrais3r

Same here, we're upgrading to node 18 and now need to --force to do any npm install...

valdiss avatar Mar 09 '23 10:03 valdiss

Looking forward to get this changes

RezaRahmati avatar Mar 20 '23 18:03 RezaRahmati

Looking forward to see this update too!

T4kumi-js avatar Apr 12 '23 13:04 T4kumi-js

@vision10 plz fix the build

jsdevel avatar Apr 14 '23 16:04 jsdevel

@jsdevel I have included changes from https://github.com/vpulim/node-soap/pull/1206 build should be ok

vision10 avatar Apr 14 '23 19:04 vision10

@jsdevel are we happy to merge now?

bughaver avatar Apr 21 '23 03:04 bughaver

is there no one able to merge here? axios is already at 1.4.0 since this pull request some of the other pull requests can be closed or just need a bit of attention

vision10 avatar Apr 28 '23 08:04 vision10

idk, this seems like a sleeping law in congress

T4kumi-js avatar Apr 28 '23 12:04 T4kumi-js

Sorry to be that guy, but could this get merged and released? It's blocking us at the moment.

colinramsay avatar May 17 '23 16:05 colinramsay

We might need to sign a petition for this PR to get merged 😆

RezaRahmati avatar May 17 '23 16:05 RezaRahmati

Agreed it would be great to see this merged

SaundersB avatar May 29 '23 14:05 SaundersB

Bump. Is anyone still maintaining this repo?

irothenbaum avatar Jun 07 '23 15:06 irothenbaum

Bump, will this be merged someday ?

Parvenu avatar Jun 19 '23 10:06 Parvenu

Bump. This is a critical update.

guilhermeledes avatar Jun 26 '23 11:06 guilhermeledes

Any update on this one @vpulim

codersbet avatar Jun 27 '23 04:06 codersbet

FYI. In order to continue to use this package we all need to now use --force or --legacy-peer-deps until this is merged. Or we will need to find an alternative. Can we maybe delegate somebody to maintain this library since there are many of us who would contribute to resolve things like this?

SaundersB avatar Jul 02 '23 15:07 SaundersB

+1

salarihamid avatar Jul 19 '23 07:07 salarihamid

+1

zindont avatar Jul 23 '23 23:07 zindont

+1

pedrokaroth avatar Jul 25 '23 10:07 pedrokaroth

sdfghujiopm

DydjyZ avatar Jul 26 '23 12:07 DydjyZ

What-the... Bump.

kenzik avatar Aug 08 '23 15:08 kenzik

Hi, why is axios a peer dependency any way? I can't see the reason. If we could move it as a normal dependency we wouldn't be here waiting for using axios 1.0 and this library together.

random42 avatar Aug 09 '23 09:08 random42

You can solve this issue without waiting for the PR to be merged, seems like this might be a dead project. In your package.json just add the following to overrides:

"dependencies": { ... }, "overrides": { "soap": { "axios": "^1.0.0" } }, "devDependencies": { ... },

RJWerning avatar Aug 11 '23 12:08 RJWerning

+1

antirek avatar Aug 26 '23 05:08 antirek

Same here! @vpulim Any chance to get this merged?

AdrianFahrbach avatar Aug 28 '23 13:08 AdrianFahrbach

Any timeline for when this pr will be merged and released ? 🤔

nicolae536 avatar Oct 05 '23 07:10 nicolae536

Merge please this is ruining the package

lukeTheWizard avatar Nov 07 '23 19:11 lukeTheWizard

We seriously need someone to accept these pull requests that are piling up, not just mine @vpulim please give someone access to manage this repository if you are too busy

axios version has known vulnerabilities https://security.snyk.io/vuln/SNYK-JS-AXIOS-6032459

vision10 avatar Nov 13 '23 15:11 vision10

@RJWerning can you confirm that the override works? I didn't want to risk it because it's a major version upgrade but now there's this new security vulnerability...

alfaproject avatar Nov 15 '23 09:11 alfaproject