digest-fetch

digest-fetch copied to clipboard

Update to ESM / node-fetch@v3

1

From the `node-fetch` [readme](https://www.npmjs.com/package/node-fetch): > `node-fetch` from v3 is an ESM-only module - you are not able to import it with `require()`. Which basically means I can't use `node-fetch@v3` and...

jaschaio

Multiple auth method support

3

Split WWW-Authenticate by challenge and handle digest-related properties only for digest method. Add tests to make sure Digest is chosen as the more secure variant.

szakharchenko

Fix arguments in parseAuth-related test

1

parseAuth expects a full WWW-Authenticate header value which includes the scheme, so it makes sense to provide it in tests.

szakharchenko

Fix async tests

1

Asynchronous tests need to run the 'done' function supplied as their argument; otherwise they are considered passing prematurely.

szakharchenko

Typescript Support

3

Any plans to add typescript support?

abraha2d

If you want to use the package in a Typescript project, without getting in trouble: you should use the following way to import it `import DigestClient = require('digest-fetch');`

CamiloPenagos99

DigestClient.prototype.fetch incorrectly overwrites headers on forwarded request

1

`DigestClient.prototype.fetch` is currently expecting its arguments to be in the form `(url: string, init: RequestInit)`, but the fetch API also supports arguments in the form of `(request: RequestInfo, init: undefined)`....

wgehring-ml

Apologies for all the changes below, vs code started formatting everything! I'm trying to add a timeout option to the fetch as I've ran into a weird networking issue calling...

coredevel