Illegal value '['

[lopezator]

We have defined some extra properties:

// API tags applied at the message level
extend google.protobuf.FieldOptions {
    mycompany.api.ResourceName resource_name = 818181;
}

And make use of them like this:

message MyMessage {
    string name = 1 [
        (mycompany.api.resource_name) = {
            any: [
                "some/*/format/*",
                "other/*/format/*"
            ]
        }
    ];
}

Given these protos, when importing them, BloomRPC throws this error:

illegal value '[' (/Users/myuser/go/src/gitlab.mycompany.com/my-apis/spec/mycompany/myproject/myfolder/v1/myproto.proto, line 100)

The only workaround I found is just dropping that part.

message MyMessage {
    string name = 1;
}

[fenos]

@lopezator Hi,
I will have a look at this, but it sounds like some sort of limitation of protobuf.js parser.

I'll let you know once i've got some more information

[lopezator]

If it were the case, maybe we should move the issue there.

Thanks @fenos!

[glerchundi]

@fenos bitten by the same, I tried to create my own dev workspace in order to see if the problem comes from protobuf-js but the steps on the CONTRIBUTING part of the README didn't work for me. Did you have time to look at this or anything I can do to push this forward?

[fenos]

@glerchundi i haven't look this yet, i'm fairy busy at the moment, but i'll get to this asap

[glerchundi]

No rush, thanks for keeping us updated!

[nickrobinson]

Running into the same issue with Version 1.4.1 (1.4.1.145)

[fenos]

Unfortunately this is a limitation of protobufjs parser, you can read more information here.

#115

I'll try to make it work in a different way in future releases if the PR doesn't get merged

[dharmjit]

Hi, facing the same issue. Any update.

[mkfsn]

Hi folks, it seems like protobufjs/protobuf.js#1256 has been merged and released, do you have any update about this issue?

[mcopjan]

any update on this please? ^^

[mcopjan]

@fenos ?

[wolfogre]

Same problem when using protoc-gen-validate, any update?

DomainKind kind = 11 [(validate.rules).enum = {in: [1,2,3]}];

[monotone]

@fenos
facing the same problem

[CyborgMaster]

Now that protobufjs/protobuf.js#1256 has been merged, it feels like the only change should be updating the package.json file. I tried to do it myself and could figure out how to get the dev environment up and running. Anyone else want to take a crack on it? Hopefully a one line change.

[monotone]

it already so long for this issue. I not good at about JS, and tried upgrade failed. Someone else could do it ？

[codewithrajranjan]

Any update of this

[kos-maverik]

Facing the same problem, is there any update on this?

[darewreck54]

Any updates on this?

[akalittle]

Any updates on this? Still facing the problem by the version 1.5.3

[wuqinqiang]

Any updates on this?

facing the same problem.😂

[M-Kepler]

Any updates on this?

facing the same problem too.