Ability to externalize WASM was broken about undici HOT 11 OPEN

I was talking about that a bit with Richard. I don't know the undici tests well enough to know off the top of my head, but building with the option enabled and validating that the wasm file is were it's expected afterwards was what we thought might work.

from undici.

Actually it just needs like 5 more lines, I promise I will provide a PR tonight at latest midnight (German Time),

from undici.

I will also provide a test.

from undici.

@Uzlopak

from undici.

There is some discussion in the PR after it landed, goal is to add back the parts that were removed but still needed while keeping what was optimized in #3074

from undici.

Yes, we discussed this, I kind of lost track of it.

from undici.

Should we revert?

@mhdawson how can we test for this in a meaningful way so we don't regress?

from undici.

@mhdawson if any of you can drop a PR for this, it would be very useful to prevent further issues.

from undici.

@Uzlopak do you mean you will include the testing in your PR?

from undici.

@Uzlopak Any updates?

from undici.

@Uzlopak just wondering if there is something I can do to help this along? If you have the right way in mind to fix but just don't have time maybe we can get together to do some knowledge transfer and either myself or @richardlau can help out?

from undici.