protobuf.js

protobuf.js copied to clipboard

Full disclosure: requesting PR for hacktoberfest, if you support it. Adding test coverage to monkey-patched implementation of deep-equal.

romanbalayan

Fails to parse "{" in options

6

protobuf.js version: 1.1.1 Fails to parse following option ```js option (scalapb.options) = { field_transformations: [ { //

kpritam-tw

protobuf.js version: 7.2.5 node v19.2.0 i have a proto test_bad.proto with two ";": ```protobuf package awesomepackage; message AwesomeMessage { optional string awesome_field = 1;; // here!!!! } ``` run the...

zxt5105515

protobuf.js version: latest Following the example in the README when sending a request it seems to be filling the buffer with trailing NULL, am I missing something? Are my encoding...

wes-kay

protobuf.js version: v7.2.5 node version: v18.17.1 npm version: v10.1.0 I'm trying to install protobuf using "npm install protobufjs" and it gives this error: ``` npm ERR! code ERR_INVALID_ARG_TYPE npm ERR!...

OriginalStefikO

bug : concurrent invocation trigger bug . (when protobuf load file to an existed root.)

1

protobuf.js version: concurrent invocation trigger bugs ```js let __protoRoot = new protobuf.Root(); function _getMessage(path,filename,cb){ var mess = __protoRoot.lookup(path); if(!mess){ protobuf.load(filename,__protoRoot,(err,root)=>{ if(err) throw err; if(!__protoRoot.nested) throw '__protoRoot.nested is not a object'...

JJ2667699

pbjs generating wrong javascript

1

protobuf.js version: master Generated javascript seems to be wrong: test.proto: ``` package awesomepackage; syntax = "proto3"; message AwesomeMessage { string awesome_field = 1; // becomes awesomeField } ``` I ran...

alanliang

CVE-2022-25878 - Fix for version 5.0.3

3

protobuf.js version: 5.0.3 I would like to fork a new branch for version 5.0.3 in order to solve CVE-2022-25878 , but I couldn't understand where the exposed code in this...

shacharkabi1

Parsing camelCase and snake_case JSON objects into protobuf.js messages

1

Google's JSON parsing functions, such as [C++'s `JsonStringToMessage()`](https://protobuf.dev/reference/cpp/api-docs/google.protobuf.util.json_util/), accept [JSON keys as camelCase](https://protobuf.dev/programming-guides/proto3/#json) even if the backing field is snake_case. By contrast, protobuf.js (AFAICT) only supports one format or the...

dimo414

undefined instead of default for optional fields ?

16

protobuf.js version: 6.10.2 Currently, to test if an optional field has been provided, you have to use `message.hasOwnProperty("optField")` because `message.optField` will return the default value (0 for numbers, `""` for...

sloonz