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

error: zlib.createBrotliDecompress is not implemented #7248

Closed
accplan opened this issue Nov 21, 2023 · 18 comments
Closed

error: zlib.createBrotliDecompress is not implemented #7248

accplan opened this issue Nov 21, 2023 · 18 comments
Labels
bug Something isn't working node.js Compatibility with Node.js APIs

Comments

@accplan
Copy link

accplan commented Nov 21, 2023

What version of Bun is running?

1.0.13+1b8d7d5cb

What platform is your computer?

Linux 5.15.0-87-generic x86_64 x86_64

What steps can reproduce the bug?

  1. Download and extract repro archive: https://file.io/cU8ltCjiKbzR
  2. bun install
  3. node server.js (server will be listening on port 3000)
  4. bun client.js

What is the expected behavior?

Console output: 200 Long text to trigger brotli compression
You can run node client.js instead to see this text

What do you see instead?

error: zlib.createBrotliDecompress is not implemented
at node:zlib:2266:64
at handleResponse (.../node_modules/axios/lib/adapters/http.js:510:25)
at .../node_modules/follow-redirects/index.js:368:4
at .../node_modules/follow-redirects/index.js:65:4
at node:http:905:28
at processTicksAndRejections (:61:76)

Additional information

Similar error will appear if you run bun server.js and try to request the content from http://127.0.0.1:3000/test.txt
error: zlib.createBrotliCompress is not implemented

@accplan accplan added the bug Something isn't working label Nov 21, 2023
@sueszli
Copy link

sueszli commented Nov 21, 2023

Having the same issue here: sueszli/notionBackup#4

@Electroid Electroid added the node.js Compatibility with Node.js APIs label Nov 21, 2023
@Binaryify
Copy link

same
v1.0.14

@Jarred-Sumner
Copy link
Collaborator

Duplicate of #267

This is being tracked in #267. Please subscribe to that issue to track the progress of Brotli support.

@Jarred-Sumner Jarred-Sumner closed this as not planned Won't fix, can't repro, duplicate, stale Nov 29, 2023
@knowhatamine
Copy link

zlib.brotliCompress is not implemented

:(

@sueszli
Copy link

sueszli commented Mar 20, 2024

to everyone that just stumbled upon this issue and is wondering what else isn't implemented: https://bun.sh/docs/runtime/nodejs-apis

@arthurwolf
Copy link

is there any way to get this to work without having to wait for a fix to get merged in ? thanks!

@arthurwolf
Copy link

@Robert27 I definitely don't think this should be closed ... it's a major issue/annoyance when trying to switch to bun...

@kingkong404
Copy link

+1 having issues still in 1.1.3

@Niklas2290
Copy link

Niklas2290 commented Apr 13, 2024

+1

@Robert27
Copy link

As said above it’s closed because it’s a duplicate of #267

@yusuf57rl
Copy link

+1

1 similar comment
@francoisrob
Copy link

+1

@SteveYuOWO
Copy link

same

@iamacup
Copy link

iamacup commented Apr 28, 2024

I've had a bunjs fargate task run daily for the past two months which is now suddently failing after a rebuild but no code change on this error in relation to Axios.

Based on the recent influx of comments on this and #267 it feels like something somewhere has changed for the worse but i'm not sure specifically what.

@iamacup
Copy link

iamacup commented Apr 28, 2024

changing FROM oven/bun:latest to FROM oven/bun:1.1.3 in my Dockerfile has removed the problem from my execution - looks like something may have changed.

Worth noting that i did not check other versions than 1.1.3, it could be that a more recent version also works, apart from latest.

@kingkong404
Copy link

changing FROM oven/bun:latest to FROM oven/bun:1.1.3 in my Dockerfile has removed the problem from my execution - looks like something may have changed.

Worth noting that i did not check other versions than 1.1.3, it could be that a more recent version also works, apart from latest.

Can confirm something in later versions of Bun has caused this to break

@yangcurve
Copy link

same

@Jarred-Sumner
Copy link
Collaborator

Jarred-Sumner commented May 5, 2024

Please follow along in #267.

This issue is being addressed in #10722, which will likely ship in Bun v1.1.8 next week.

I am locking this because this is a duplicate of #267 and that is the right place to follow along

@oven-sh oven-sh locked and limited conversation to collaborators May 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working node.js Compatibility with Node.js APIs
Projects
None yet
Development

No branches or pull requests