Robert Nagy

icon indicating a website link https://www.npmjs.com/~ronag icon indicating an email [email protected]

icon company @nxtedition icon location Malmö icon location Co-Founder @nxtedition Maintainer @CasparCG Collaborator & TSC Member @nodejs

Results 722 comments of Robert Nagy

Semver Major Considerations

Once you are done with that I have some ideas I'd like to experiment to further simplify and improve the dispatch api.

Semver Major Considerations

@metcoder95 another step we should do is to change the handlers (i.e. retry, redirect etc...) to export functions instead of classes, i.e. ```js export const redirect = (dispatch) => ({...

Semver Major Considerations

I don't have a clear timeline atm. I don't think these changes should be noticable inside node.

Client network socket disconnected before secure TLS connection was established

What is the error? msg? stack?

[http.IncomingMessage] `req.toWeb()`

Doesn't the alternative you propose already work?

[http.IncomingMessage] `req.toWeb()`

What about just introducing a new api? To avoid confusion and overlap, i.e.: ```js await http.serve(async req => { req.respondWith(fetch('https://httpbin.org/get')) }, { port }) ``` https://deno.land/[email protected]/examples/http_server#using-the-stdhttp-library

[http.IncomingMessage] `req.toWeb()`

I think the biggest challenge is implementing `req`.

[http.IncomingMessage] `req.toWeb()`

Actually my previous example was wrong. Should be: ```js await http.serve(async req => { return fetch('https://httpbin.org/get') }, { port }) ``

[http.IncomingMessage] `req.toWeb()`

@jimmywarting where does the `responseWith` come from?

[http.IncomingMessage] `req.toWeb()`

Maybe this should be moved to wintercg?