Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deno hangs after succesfully executing npm-package library call #23727

Open
aukeroorda opened this issue May 7, 2024 · 0 comments
Open

Deno hangs after succesfully executing npm-package library call #23727

aukeroorda opened this issue May 7, 2024 · 0 comments
Labels
bug Something isn't working node compat

Comments

@aukeroorda
Copy link

> deno --version
deno 1.41.3 (release, x86_64-apple-darwin)
v8 12.3.219.9
typescript 5.3.3

MRE

Can be found at: https://github.com/aukeroorda/mre-deno-hang/tree/main

  1. Verify the contents of the repo
  2. Install the paper-js library as specified in the package file:
    npm install
  3. Run deno:
    deno run mre.ts
  4. Allow reading of node_modules: y
  5. Observe that code executes to the end of the mre.ts script, but Deno hangs.

Note:
Deno doesn't hang anymore if the following lines 20 and 21 are commented out: https://github.com/aukeroorda/mre-deno-hang/blob/6c3d8d0f313141bff39ffc0906780189668b055d/mre.ts#L20-L21

Inspecting using chromium

I've tried to inspect what is going on with this library call using the chromium debugger, but I can't really see much, as the call gets 'hijacked' by a call to node:timers setInterval() function, and I lose the trace of the library function call here.

@aukeroorda aukeroorda changed the title Deno hangs after specific library call Deno hangs after succesfully executing npm-package library call May 7, 2024
@bartlomieju bartlomieju added bug Something isn't working node compat labels May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working node compat
Projects
None yet
Development

No branches or pull requests

2 participants