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

Feat: bun support #352

Merged
merged 19 commits into from
Oct 30, 2023
Merged

Feat: bun support #352

merged 19 commits into from
Oct 30, 2023

Conversation

akvlad
Copy link
Collaborator

@akvlad akvlad commented Oct 6, 2023

bun support

Breaking changes

  1. qryn.js is renamed to qryn.mjs according to the node standards.
  2. Node versions with no ES6 support (I believe v14) are deprecated

Known issues

Sometimes during the e2e test I have experienced the bun crash with the message of pure virtual function call:

pure virtual method called

uh-oh: A C++ exception occurred
bun will crash now 😭😭😭

----- bun meta -----
Bun v1.0.1 (31aec4eb) Linux x64 #34-Ubuntu SMP PREEMPT_DYNAMIC Mon Sep  4 13:06:55 UTC 2023
AutoCommand: 
Elapsed: 449470ms | User: 5805ms | Sys: 1363ms
RSS: 0.20GB | Peak: 4.78GB | Commit: 0.20GB | Faults: 0
----- bun meta -----

0   0x55f268efa3bb
1   ???
2   ???
3   ???
4   ???
5   ???
6   ???
7   ???
8   ???
9   ???
10  ???
11  ???
12  ???
13  ???
14  ???
15  ???
16  ???
17  ???
18  ???
19  ???
20  ???
21  ???
22  ???
23  ???
24  ???
25  ???

Crash report saved to:
  ~/.bun-crash/v1.0.1-1696593282967.crash

The crash is observed under different circumstances.
This should be avoided via running qryn with pm2 or docker with auto-restart.
The community help would be nice.

UPD: the issue seems fixed. The Bun worker module is not production ready. Will check for several next hours.

@socket-security
Copy link

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
@stricjs/router 4.2.9 None +0 35.2 kB reve-node

@lmangani lmangani changed the title #feat: bun support Feat: bun support Oct 6, 2023
lmangani and others added 14 commits October 6, 2023 18:29
Signed-off-by: Lorenzo Mangani <[email protected]>
- Releases synced with Tags
- Only publish upon new release

Signed-off-by: Lorenzo Mangani <[email protected]>
Signed-off-by: Lorenzo Mangani <[email protected]>
Signed-off-by: Lorenzo Mangani <[email protected]>
Signed-off-by: Lorenzo Mangani <[email protected]>
Signed-off-by: Lorenzo Mangani <[email protected]>
Signed-off-by: Lorenzo Mangani <[email protected]>
Signed-off-by: Lorenzo Mangani <[email protected]>
Signed-off-by: Lorenzo Mangani <[email protected]>
@akvlad akvlad merged commit 79c374c into master Oct 30, 2023
6 checks passed
@akvlad akvlad deleted the bun_support branch October 30, 2023 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants