Carl Ansley
Carl Ansley
What would you propose for default functionality for HEAD? Do you see an issue with the current behavior?
We use HEAD quite a bit in our REST APIs, it has specific semantics that are useful, so don't agree it's simply a middleware thing. But I'm fine with your...
swagger2 doesn't currently support binary files/attachments. I'm not sure it really provides any value in this situation since it can't really validate anything. But PRs welcome if you want to...
Fetch is a standard Web API, now included in Node (and almost all modern Javascript runtimes). I understand it's more performant than the Node http request implementation, and we're standardizing...
@RanVaknin I think you are misunderstanding the problem here. This issue is not about re-writing `node-http-handler` to use fetch, breaking old versions of Node, etc. This is about fixing the...
In the current version of Node (v22), `fetch` is stable: https://nodejs.org/api/globals.html#fetch
> Hi @carlansley, > > > This issue is not about re-writing node-http-handler to use fetch, breaking old versions of Node, etc. > > Your request is to rewrite _**fetch**_-http-handler...
I can confirm `FetchHttpHandler` now works in Node 20.13.1 and 22.1.0 with version 3.575 of the SDK. @kuhe thanks for working on this and providing excellent documentation regarding the current...
Go for it! Let me know if you have any questions/issues.
I can have a look at doing the test(s) if you feel your implementation is in a decent state. Regarding the deref, I agree I'd like it moved out of...