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

latest release error - RangeError: "length" is outside of buffer bounds #54523

Closed
DevboiDesigns opened this issue Aug 23, 2024 · 4 comments · Fixed by #54524
Closed

latest release error - RangeError: "length" is outside of buffer bounds #54523

DevboiDesigns opened this issue Aug 23, 2024 · 4 comments · Fixed by #54524
Labels
duplicate Issues and PRs that are duplicates of other issues or PRs.

Comments

@DevboiDesigns
Copy link

DevboiDesigns commented Aug 23, 2024

Version

node:latest

Platform

macOS Sonoma 14.6.1
23.6.0 Darwin Kernel Version 23.6.0: Mon Jul 29 21:14:30 PDT 2024; root:xnu-10063.141.2~1/RELEASE_ARM64_T6000 arm64

Subsystem

No response

What steps will reproduce the bug?

releasing docker image with node:latest to Heroku and firestore configured

How often does it reproduce? Is there a required condition?

every time it tries to run

new FirestoreClient (/app/node_modules/@google-cloud/firestore/build/src/v1/firestore_client.js:136:32)

What is the expected behavior? Why is that the expected behavior?

App should just start

What do you see instead?

It crashes with warning

2024-08-23T10:31:33.265229+00:00 app[web.1]: RangeError: "length" is outside of buffer bounds
2024-08-23T10:31:33.265230+00:00 app[web.1]: at Buffer.proto.utf8Write (node:internal/buffer:1066:13)
2024-08-23T10:31:33.265230+00:00 app[web.1]: at Op.writeStringBuffer [as fn] (/app/node_modules/protobufjs/src/writer_buffer.js:61:13)
2024-08-23T10:31:33.265231+00:00 app[web.1]: at BufferWriter.finish (/app/node_modules/protobufjs/src/writer.js:453:14)
2024-08-23T10:31:33.265231+00:00 app[web.1]: at /app/node_modules/@grpc/proto-loader/src/index.ts:382:62
2024-08-23T10:31:33.265231+00:00 app[web.1]: at Array.map (<anonymous>)
2024-08-23T10:31:33.265232+00:00 app[web.1]: at createPackageDefinition (/app/node_modules/@grpc/proto-loader/src/index.ts:381:47)
2024-08-23T10:31:33.265232+00:00 app[web.1]: at Object.fromJSON (/app/node_modules/@grpc/proto-loader/src/index.ts:453:10)
2024-08-23T10:31:33.265232+00:00 app[web.1]: at GrpcClient.loadProtoJSON (/app/node_modules/google-gax/build/src/grpc.js:228:51)
2024-08-23T10:31:33.265233+00:00 app[web.1]: at new LocationsClient (/app/node_modules/google-gax/build/src/locationService.js:118:32)
2024-08-23T10:31:33.265233+00:00 app[web.1]: at new FirestoreClient (/app/node_modules/@google-cloud/firestore/build/src/v1/firestore_client.js:136:32) {
2024-08-23T10:31:33.265234+00:00 app[web.1]: code: 'ERR_BUFFER_OUT_OF_BOUNDS'
2024-08-23T10:31:33.265234+00:00 app[web.1]: }

Additional information

I set my node version to node:20 and this resolves it for now

@nodejs nodejs deleted a comment Aug 23, 2024
@RedYetiDev
Copy link
Member

Duplicate of #54521 and #54518

@RedYetiDev RedYetiDev closed this as not planned Won't fix, can't repro, duplicate, stale Aug 23, 2024
@RedYetiDev RedYetiDev added the duplicate Issues and PRs that are duplicates of other issues or PRs. label Aug 23, 2024
ronag added a commit to nxtedition/node that referenced this issue Aug 23, 2024
ronag added a commit to nxtedition/node that referenced this issue Aug 23, 2024
ronag added a commit to nxtedition/node that referenced this issue Aug 23, 2024
ronag added a commit to nxtedition/node that referenced this issue Aug 23, 2024
ronag added a commit to nxtedition/node that referenced this issue Aug 23, 2024
ronag added a commit to nxtedition/node that referenced this issue Aug 23, 2024
@jakecastelli jakecastelli reopened this Aug 23, 2024
@jakecastelli
Copy link
Member

This issue is indeed duplicate of #54518, since the fix PR also links to this issue, I will let the Github to auto close this issue once the fix lands.

@jakecastelli
Copy link
Member

Actually, we should probably ask Robert to amend the commit message to link to the other issue instead. Sorry for the confusion.

@RedYetiDev RedYetiDev closed this as not planned Won't fix, can't repro, duplicate, stale Aug 23, 2024
@jakecastelli
Copy link
Member

Thanks @RedYetiDev!

ronag added a commit to nxtedition/node that referenced this issue Aug 26, 2024
nodejs-github-bot pushed a commit that referenced this issue Aug 27, 2024
Fixes: #54523
Fixes: #54518
PR-URL: #54524
Reviewed-By: Ethan Arrowood <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
RafaelGSS pushed a commit that referenced this issue Aug 30, 2024
Fixes: #54523
Fixes: #54518
PR-URL: #54524
Reviewed-By: Ethan Arrowood <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
RafaelGSS pushed a commit that referenced this issue Aug 30, 2024
Fixes: #54523
Fixes: #54518
PR-URL: #54524
Reviewed-By: Ethan Arrowood <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
louwers pushed a commit to louwers/node that referenced this issue Nov 2, 2024
Fixes: nodejs#54523
Fixes: nodejs#54518
PR-URL: nodejs#54524
Reviewed-By: Ethan Arrowood <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate Issues and PRs that are duplicates of other issues or PRs.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants
@RedYetiDev @jakecastelli @DevboiDesigns and others