@swissspidy: I thought I didn't, but now after some investigation I found out I indeed did. :)

For this ticket: there shouldn't be double slashes in URLs, but was that stated in dev notes? I guess this doesn't need to be changed on the Core side, but if it hasn't been posted yet, a note would be helpful.

I was going to hold off on a 4.6.1 commit here, as we might be able to just fix it properly ​within Requests, however it looks like that fix may not drop until next week now, so I expect trunk will get a proper fix for this, but 4.6.x can just keep this regex.

I believe the goal here was to strip // from the path of URLs down to a single / however, it's also affecting any query string parameters, as seen in this example from the EDD Software Licensing parameters sent:

@cklosows Thanks for the report. You're correct in that other double slashes should not be stripped. As this ticket is closed against the 4.6.1 milestone, can you open a new ticket with that information so we can target for 4.6.2?

HTTP: Update Requests to master (0048f3c) which fixes a number of outstanding issues.

Fixes #38070, #37733 by reverting part of [38429] and using the fix in Requests.
Fixes #37992 allowing for connecting to SSL resources on ports other than 443.
Fixes #37991 by not sending default ports in the Host: header.
Fixes #37839 to match and decode Chunked responses correctly.
Fixes #38232 allowing a SSL connection to ignore the hostname of the certificate when verification is disabled.

Fixes #38070, #37733 by reverting part of [38429] and using the fix in Requests.
Fixes #37992 allowing for connecting to SSL resources on ports other than 443.
Fixes #37991 by not sending default ports in the Host: header.
Fixes #37839 to match and decode Chunked responses correctly.
Fixes #38232 allowing a SSL connection to ignore the hostname of the certificate when verification is disabled.