Cannot read property 'root' of undefined error about bloomrpc HOT 4 CLOSED

I see, I'm primarily using grpc-java, so dots in the package name are quite common :).

After removing the package name, it seems to load proto now. However, when going into my "put" endpoint I see the following.

{
  "error": "Error parsing the request message, please report the problem sharing the offending protofile"
}

This is with the following modified proto

syntax = "proto3";

import "google/api/annotations.proto";

package comsomeprofile;

service UserDataApi {
    rpc get(ReadRequest) returns (ReadResponse) {
        option (google.api.http) = {
        get: "/v1/read"
      };
    }

    rpc put(WriteRequest) returns (WriteResponse) {
        option (google.api.http) = {
        put: "/v1/write"
        body: "*"
      };
    }
}

message ReadRequest {
    repeated string userIds = 1;
    repeated string fields = 2;
}

message ReadResponse {
    map<string, InternalData> internalData = 1;
    map<string, ThirdPartyData>  thirdPartyData = 2;
}

message WriteRequest {
    map<string, InternalData> internalData = 1;
    map<string, ThirdPartyData> thirdPartyData = 2;
    int64 ttl = 3;
}

message WriteResponse {
    string userId = 1;
    bool status = 2;
}

message InternalData {
    map<string, string> data = 1;
}

message ThirdPartyData {
    map<string, Segments> data = 1;

    message Segments {
        repeated string segments = 1;
    }
}

Let me know if I should open this up another issue for this.

from bloomrpc.

@yeenow123 👆 It will fix both of your issues! I'm preparing the release

from bloomrpc.

Hi,

This is an interesting one, I think it's an edge case not covered in the editor.

From the first look at the protofile, I'm guessing that the offending line is the package name having a dot in the name.

Try to remove the dot from the package and try to import it again.

Thanks for reporting this, i'll be fixing this asap

from bloomrpc.

I think those are 2 slightly different problems, they will be both fixed.

Thanks for sharing this, it helps a lot to fix the parsing issue.

As I mentioned there is some issue parsing this specific protofile, I probably guess that I forgot to cover the map data type. I'll be confirming this once I see the stack trace.

Apologize for the inconvenient, I'll be trying to push a fix this evening!

from bloomrpc.