You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
According to the v8 documentation, this indicates that threading use for V8 may not be set up correctly. I was uncertain whether to file this as a bug in ssr-rs or tuono, but I've filed it here as it can be reproduced from tuono.
Description
The tuono server can segfault during rendering of a second response when built for release.
NB: A system with the PKU CPU feature may be needed to reproduce the segfault.
Expected behaviour
It is expected that the server will not segfault.
How to reproduce
Nix is required for easy reproduction, see: https://nixos.org/download/
curl http://localhost:3000
multiple times. The segfault should occur on the second request.Notes
v8::Isolate::new
withinssr-rs
during the initialization ofProdJs::SSR
intuono
.Fix
Making the following update to
ssr-rs
removes the segfault:https://github.com/Valerioageno/ssr-rs/blob/main/src/ssr.rs#L52
According to the
v8
documentation, this indicates that threading use for V8 may not be set up correctly. I was uncertain whether to file this as a bug inssr-rs
ortuono
, but I've filed it here as it can be reproduced from tuono.https://docs.rs/v8/134.1.0/v8/fn.new_unprotected_default_platform.html
Relevant issue: denoland/rusty_v8#1381
Screenshots
No response
System Info
System: OS: Linux 6.6 NixOS 24.05 (Uakari) 24.05 (Uakari) CPU: (20) x64 13th Gen Intel(R) Core(TM) i7-1370P Memory: 37.89 GB / 62.45 GB Container: Yes Shell: 5.2.37 - /nix/store/p6k7xp1lsfmbdd731mlglrdj2d66mr82-bash-5.2p37/bin/bash Binaries: Node: 20.18.1 - /nix/store/wfxq6w9bkp5dcfr8yb6789b0w7128gnb-nodejs-20.18.1/bin/node Yarn: 1.22.22 - /nix/store/jcgdksj946l5l42c2y9ks2l4g6n74h3f-yarn-1.22.22/bin/yarn npm: 10.8.2 - /nix/store/wfxq6w9bkp5dcfr8yb6789b0w7128gnb-nodejs-20.18.1/bin/npm pnpm: 9.14.4 - /nix/store/rzxqj4y899lcr3c2rx92pf457dk6086w-pnpm-9.14.4/bin/pnpm npmPackages: tuono: link:../../packages/tuono => 0.17.8
System info (Rust)
Additional context
PKU may be required to reproduce the issue (listed in CPU flags as
pku
):The text was updated successfully, but these errors were encountered: