supabase / edge-runtime

A server based on Deno runtime, capable of running JavaScript, TypeScript, and WASM services.
MIT License
689 stars 62 forks source link

fix: inspector causes panic #404

Closed nyannyacha closed 2 months ago

nyannyacha commented 2 months ago

What kind of change does this PR introduce?

Bug fix

Description

It seems that the path to calling thread::unpark was enabled when eszip content became possible to load asynchronously.

The current inspector implementation upstream doesn't like it when park is called at a certain stage of poll_session and then immediately unparked. This results in reaching unreachable!, which causes panic.

Related Links

Reported-by: @daniel-sc

laktek commented 2 months ago

Squash and merge please

github-actions[bot] commented 2 months ago

:tada: This PR is included in version 1.58.2 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

github-actions[bot] commented 2 months ago

:tada: This PR is included in version 1.58.2 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: