Improve support for pre-shared sessions (PSK) about undici HOT 3 CLOSED

Is there a reason we spread these options first instead of last? I'd expect the module should prioritize the passed settings over anything internal by default in most cases.

as you can see, in that example, we are reading the session from the cache. I guess this was added without thinking of the preshared key.

A PR that improves this would be great.

from undici.

I don't mind opening a PR for this, but think it first needs some discussion/review to make sure we are aligned on the path forward.

Also side note I noticed the type for factory options is object, which made this a bit confusing at first when I thought I could simply inject my session on the pool factory only to discover the connect type is actually a function. It'd be great to improve that type.

from undici.

@mcollina opened a draft PR to simply spread options at the bottom, so anything can be overriden. Do you think this is okay or was it intentional somethings like socket and ALPNProtocols are set by the module?

from undici.