mirror of
https://github.com/ProxymanApp/Proxyman.git
synced 2026-04-26 08:35:53 +03:00
[GH-ISSUE #1239] Protobuf not working properly #1234
Labels
No labels
Discussion
Feature request
In Progress...
Plugins
Waiting response
Windows
Windows
bug
duplicate
enhancement
feature
good first issue
iOS
macOS 10.11
question
wontfix
✅ Done
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/Proxyman#1234
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @kobe24u on GitHub (May 18, 2022).
Original GitHub issue: https://github.com/ProxymanApp/Proxyman/issues/1239
Originally assigned to: @NghiaTranUIT on GitHub.
Proxyman version? (Ex. Proxyman 1.4.3)
3.5.0
macOS Version? (Ex. mac 10.14)
Mac 12.3.1
Steps to reproduce
Hey I imported all the necessary .proto files but the response still contains a lot of numbers, and looks like it lacks some message types compared to Charles, my company advises us to use Charles with the protobuf descriptor file imported, just wondering do we have any plan to support protobuf descriptor file some day in the future? thanks
Expected behavior
Screenshots (optional)
@NghiaTranUIT commented on GitHub (May 20, 2022):
Thanks for the report @kobe24u.
Currently, Proxyman did support all common Google Protobuf types. It seems to be a bug if it didn't resolve the protobuf message properly. I guess using the protobuf descriptor file as Charles does might fix the problem (Currently, Proxyman imports .proto files).
I will open this ticket and soon implement it 👍
@kobe24u commented on GitHub (May 21, 2022):
thanks @NghiaTranUIT , that's great news!
@kobe24u commented on GitHub (Jun 18, 2022):
Hello @NghiaTranUIT hope you are doing well, just want to check is this still on your radar? My company is seriously considering buying a team license and widely adopt it, in our case, the only thing Proxyman lacks is this Protobuf descriptor file support, so we are really looking forward to a new version that supports this, just keen to know if there is any ETA of this feature? Or is there any difficulty adding this and currently no ETA?
Thank you and hope you are having a good day :)
@NghiaTranUIT commented on GitHub (Jun 19, 2022):
Thanks for letting me know your situation. I did a quick fix and turn out the problem is how Proxyman implemented the Protobufs.
Using the schema file might have some problems, which you already mentioned (missing some column fields). To fix it, we have to drop the Schema file and support FileDescriptorSet.
The ETA is not set yet, since we have different high-priority tasks : <
@NghiaTranUIT commented on GitHub (Jun 19, 2022):
I will reschedule it later after 2 release versions. See if I can support the FileDescriptorSet 👍
ETA ~ 2-3 months
@kobe24u commented on GitHub (Jun 20, 2022):
Thank you very much @NghiaTranUIT , really appreciate for the update and can't wait to try the new app 👍 😄
@NghiaTranUIT commented on GitHub (Jun 22, 2022):
Hey @kobe24u let try this Beta build: https://proxyman.s3.us-east-2.amazonaws.com/beta/Proxyman_3.5.2_Support_Desc_file_for_Protobuf.dmg
I finally support the Protobuf descriptor file (*.desc) 👍
It's better than *.proto since you can import the desc with any order. You can also merge them all into a single desc file too.
Here is the script to generate a desc file (if you don't have one): https://docs.proxyman.io/advanced-features/protobuf#2.-protobuf-file-descriptor-.desc
Make sure you import your
descfile into the Schema List without any error. If there is any error, please report to me 😄@kobe24u commented on GitHub (Jun 22, 2022):
Hi Nghia, thanks so much for this update, I just gave it a crack, and for
some reason it doesn't work as expected, I imported the .desc file and I
can see some console log,and when I try to view the response I can see the
message type has been filled in in the drop down menu which is great, I
selected the message type and press done, but got an error message saying
[ERROR] Invalid configuration. Could not find the type: type.googleapis.com/google.protobuf.Any, which is something I don't getwhile using Charles, I've provided some screenshots for your reference.
Regards
[image: 2022-06-22_22-57-08.png][image: Screen Shot 2022-06-22 at
22.55.46.png][image: Screen Shot 2022-06-22 at 22.56.21.png]
On Wed, 22 Jun 2022 at 22:41, Nghia Tran @.***> wrote:
@kobe24u commented on GitHub (Jun 22, 2022):
Just attached the screenshots for your reference. thanks
@NghiaTranUIT commented on GitHub (Jun 22, 2022):
Thanks for quick response @kobe24u . If you don't mind, please export the Protobuf request as a ProxymanLog, and the universallist.desc file to support@proxyman.io (Your data is protected and doesn't share to any 3rd-party)
We'd like to investigate what the problem is and get back to you soon/
@kobe24u commented on GitHub (Jun 22, 2022):
Ok @NghiaTranUIT I think I fixed the issue myself, if we open Charles, we can it has already imported

google.protobuf.descfor us, and I think that's something Proxyman lacks, I've seen the description sayingProxyman automatically imports all common types from Google Protobuf, such as Timestamp, Struct, Value, Enum, Method, etc.But I think it does not work properly,
I initially got the error saying

[ERROR] Invalid configuration. Could not find the type: type.googleapis.com/google.protobuf.Any, so I Downaded theProtobuf Buffers V21.1source code from here, it does not include the descriptor file, so I have to generate one myself, and for some reason, the command you provided does not work, see screenshot attached below, I have to generate one by one, glad in my case, I only needany.protoandwrappers.proto,but finally I got it working, I would be very happy if the google common types can be supported properly so we don't have to import ourselves, also worth checking if the descriptor file generator command is working on your side
Cheers
@NghiaTranUIT commented on GitHub (Jun 23, 2022):
Thanks for your investigation. I help me a lot 🙌
Speaking of
*.proto, it looks like the current directory must be at/Users/vinnie.liu/Desktop/input. To fix it, it's:Ref: https://stackoverflow.com/a/24965532/3127477
I will try to reproduce the Unknown issue 👍
@kobe24u commented on GitHub (Jun 23, 2022):
hmmmm, still not working properly, but yes it will be good if the google common types are already there. :semi
@NghiaTranUIT commented on GitHub (Jun 23, 2022):
@kobe24u let try this build: https://proxyman.s3.us-east-2.amazonaws.com/beta/Proxyman_3.5.2_Update_new_google_desc.dmg
I've updated all google pro files again. Hopefully, it fixes your case.
@kobe24u commented on GitHub (Jun 23, 2022):
@NghiaTranUIT You are a legend! It's working now! Thanks so much for getting this fixed so quickly!
@NghiaTranUIT commented on GitHub (Jun 23, 2022):
Awesome. I will wrap up and ship it in the next release soon 👍