[GH-ISSUE #342] HTTPS enabled proxying removes whitespace percent encoding from URL Paths #338

Closed
opened 2026-03-03 19:17:35 +03:00 by kerem · 6 comments
Owner

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

  1. Enable HTTPS Proxying for a domain
  2. Make request with the path containing a whitespace, e.g. https://example.com/products/Product%20Name
  3. Observe the request URL is rewritten to https://example.com/products/Product Name and will be handled differently on the server compared to a normal %20 encoding 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.

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 1. Enable HTTPS Proxying for a domain 2. Make request with the path containing a whitespace, e.g. `https://example.com/products/Product%20Name` 3. Observe the request URL is rewritten to `https://example.com/products/Product Name` and will be handled differently on the server compared to a normal `%20` encoding 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.
kerem 2026-03-03 19:17:35 +03:00
Author
Owner

@NghiaTranUIT commented on GitHub (Dec 2, 2019):

Thank for the report. Let me investigate and sent you a hot fix soon 👍

<!-- gh-comment-id:560387912 --> @NghiaTranUIT commented on GitHub (Dec 2, 2019): Thank for the report. Let me investigate and sent you a hot fix soon 👍
Author
Owner

@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 👍

<!-- gh-comment-id:560429649 --> @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 👍
Author
Owner

@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 🌮

<!-- gh-comment-id:560430008 --> @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 🌮
Author
Owner

@joshuafinch commented on GitHub (Dec 3, 2019):

Thank you @NghiaTranUIT, that appears to be working for us now!

<!-- gh-comment-id:561083624 --> @joshuafinch commented on GitHub (Dec 3, 2019): Thank you @NghiaTranUIT, that appears to be working for us now!
Author
Owner

@NghiaTranUIT commented on GitHub (Dec 3, 2019):

Glad to know that work for you 🎉 I will release officially the fix in this week

<!-- gh-comment-id:561108764 --> @NghiaTranUIT commented on GitHub (Dec 3, 2019): Glad to know that work for you 🎉 I will release officially the fix in this week
Author
Owner

@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 🐛

<!-- gh-comment-id:561717328 --> @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 🐛
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/Proxyman#338
No description provided.