mirror of
https://github.com/ProxymanApp/Proxyman.git
synced 2026-04-26 16:45:57 +03:00
[GH-ISSUE #342] HTTPS enabled proxying removes whitespace percent encoding from URL Paths #338
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#338
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 @joshuafinch on GitHub (Dec 2, 2019).
Original GitHub issue: https://github.com/ProxymanApp/Proxyman/issues/342
Originally assigned to: @NghiaTranUIT on GitHub.
Proxyman version? (Ex. Proxyman 1.4.3)
Proxyman 1.9.3
macOS Version? (Ex. mac 10.14)
macOS 10.15.1
Steps to reproduce
https://example.com/products/Product%20Namehttps://example.com/products/Product Nameand will be handled differently on the server compared to a normal%20encoding for whitespace in endpoint paths.Expected behavior
URL to not be rewritten, and to not remove the provided percent encoding when performing requests with HTTPS proxying enabled for a domain.
@NghiaTranUIT commented on GitHub (Dec 2, 2019):
Thank for the report. Let me investigate and sent you a hot fix soon 👍
@NghiaTranUIT commented on GitHub (Dec 2, 2019):
Hey @joshuafinch, @zagahr, Here is the Beta fix: https://proxyman.s3.us-east-2.amazonaws.com/beta/Proxyman_1.9.3_Percent_encoded_path.dmg
Basically, we accidentally remove the percent encoded in the path. We fixed it and will officially release in this week 👍
@NghiaTranUIT commented on GitHub (Dec 2, 2019):
In the meantime, I would keep all Percent-encoded in URL and query in entire the app too, because it makes sense and easier to deal with when we repeat or compose new URL 🌮
@joshuafinch commented on GitHub (Dec 3, 2019):
Thank you @NghiaTranUIT, that appears to be working for us now!
@NghiaTranUIT commented on GitHub (Dec 3, 2019):
Glad to know that work for you 🎉 I will release officially the fix in this week
@NghiaTranUIT commented on GitHub (Dec 4, 2019):
Proxyman 1.10.0 is out, which includes this fix too 👍 https://github.com/ProxymanApp/Proxyman/releases/tag/1.10.0
Thank you for raising the bug 🐛