supabase / edge-runtime

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

fix: mitigate denoland/deno_core#762 #377

Closed nyannyacha closed 4 months ago

nyannyacha commented 4 months ago

What kind of change does this PR introduce?

Bug fix

Description

There are cases where the specifier is mapped to a different alias by the module loader.

In this case, the waker for the future that resolves the module in the upstream codebase may not be properly registered, causing the module resolution phase to hang indefinitely.

This upstream bug has been fixed in denoland/deno_core#762, but since we don't plan to version up the Deno codebase yet, we decided to fork and cherry-pick that PR.

Related supabase/supabase#26840

nyannyacha commented 4 months ago

cc @laktek

https://github.com/nyannyacha/edge-runtime/blob/92119aebfc7c0fb82669abfc32d1804d6554fde9/Cargo.toml#L85-L86

github-actions[bot] commented 4 months ago

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

The release is available on GitHub release

Your semantic-release bot :package::rocket: