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

Feature request: support for brotli #3

Open
M-Gonzalo opened this issue Sep 28, 2023 · 3 comments
Open

Feature request: support for brotli #3

M-Gonzalo opened this issue Sep 28, 2023 · 3 comments

Comments

@M-Gonzalo
Copy link

Any plans to add brotli to the mix? It's the most efficient code to date, and widely supported in all modern clients.
Thanks in advance!

@Gusb3ll
Copy link
Owner

Gusb3ll commented Sep 28, 2023

Seem like brotliCompressSync hasn't been implemented on bun yet so I'm looking for a workaround rn

SyntaxError: Import named 'brotliCompressSync' not found in module 'node:zlib'

@M-Gonzalo
Copy link
Author

Yes, it might take using an external binary but iDK if bun supports that type of module installation. I never could install anything that needs a compiling phase.
Thanks for looking into it though. Let me know if I can help in anything towards this goal.

@Gusb3ll
Copy link
Owner

Gusb3ll commented Sep 29, 2023

Yes, it might take using an external binary but iDK if bun supports that type of module installation. I never could install anything that needs a compiling phase. Thanks for looking into it though. Let me know if I can help in anything towards this goal.

Aight, I've already created a feature request on bun rn, oven-sh/bun#6152

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

No branches or pull requests

2 participants