protobuf.js issues

Generate js and d.ts files with imports. Fix creation of nested namespaces with the same names.

1

Fixed two issues. 1) If protofile uses some Namespace (for example, Test) and it contains some Service with the same name, nested Namespace Test.Test will be generated. This Namespace will...

AVKurbatov

Prototype Pollution vulnerability

3

protobuf.js version: 7.0.0 - 7.2.4 Just including the package to trigger the vulnerability warning. This is related to https://github.com/advisories/GHSA-h755-8qp9-cq85 ``` functions %> npm audit fix up to date, audited 1215...

cupidchan

npm exec `could not determine executable to run`

protobuf.js version: 7.3.0 npm version 10.8.1 When attempting to run via `npm exec`, I'm getting an error message: ```console > npx protobufjs npm error could not determine executable to run...

rotu

C++ and JS compatability

2

protobuf.js version: 7.3.0, C++ protobuf version 21.4 Is it acutally possible to decode cpp-encoded protobuf objects with protobufs.js 7.3.0 and vice versa? Currently C++ protobuf version 21.4 and protobuf.js version:...

SvenGebert

Is the example rpcImpl in the README.md correct?

8

The example given is: ``` const rpcImpl = function(method, requestData, callback) { client.makeUnaryRequest( method.name, arg => arg, arg => arg, requestData, callback ) } ``` That gives a method call...

GeorgeBills

fix!: nested namespace resolution

2

When resolving types, we shouldn't check nested namespaces. Example: ```proto message Foo { required Bar b = 1; required Qux q = 2; message Bar { required Corge c =...

N2D4