telerik / fiddler-everywhere

This public repository aims at providing a convenient way for users of Fiddler Everywhere to report issues they may encounter while using the tool.
8 stars 0 forks source link

Protobufs that `import` from a "base dir" break #31

Open lunawinxp opened 1 month ago

lunawinxp commented 1 month ago

Describe the bug If you try to import a .proto in the Protobuf section of the settings window, that is structured like:

protos_folder/
    subFolder/
        protoFile1.proto
    subFolder2/
        protoFile2.proto

Where protoFile1.proto would contain something like import "subFolder2/protoFile2.proto"; If you would try to load this proto into Fiddler Everywhere, it would look for "subFolder2/protoFile2.proto" in the current directory of protoFile1 Fiddler Everywhere tries to load this like protos_folder/subFolder/subFolder2/protoFile2.proto

We need the ability to be able to set a "base directory", so Fiddler Everywhere knows where to find these import files

Also, For projects with hundreds of proto files, its not a trivial task to copy each one to a different folder, fix the imports to work with a static path, and then load each one in manually to the protobuf tab. The ability to add multiple proto files (or folder with subfolders included automatically) would be incredibly handy, but that's a feature request.

To Reproduce Steps to reproduce the behavior:

  1. Click settings
  2. Click Protobuf on the left
  3. Add a protobuf with the structure outlined above ^
  4. See error

image

image

NickIliev commented 1 month ago

@lunawinxp, thank you for bringing that one to our attention!

Indeed, there is no option to provide a base dir at this moment, but that sounds like a nice feature. I am marking this one as an enhancement feature, and we will let you know once the team decides on how to proceed with this type of Protobuf structure.