Is there a plan to add options for setting HTTP headers on EventSource?

[Uzlopak]

Discussed in #3112

^{Originally posted by zbinlin April 13, 2024}
Likes Websocket, we can adds a dispatcher option to EventSource constructor.

[Uzlopak]

@KhafraDev

Your opinion on this request?

[KhafraDev]

we should add it

[mertcanaltin]

I added a dispatcher in EventSource, how should I manage it, do we have certain rules? @Uzlopak @KhafraDev

[KhafraDev]

The same way fetch and websocket implement it https://github.com/nodejs/undici?tab=readme-ov-file#undicifetchinput-init-promise

[mertcanaltin]

I wonder if that's a good method
mertcanaltin@8f6cf8b

[KhafraDev]

No, dispatcher should be added as an optional property in the eventsource options (second parameter) and I'm not sure if we need to add custom headers since it can be done with Agent. Also there is no step 11 that adds custom headers in the spec so all of those changes to comments should be undone.