proxygen icon indicating copy to clipboard operation
proxygen copied to clipboard

add `URL::join(string_view relative)` ?

Open kvtb opened this issue 2 years ago • 1 comments

It would be very convenient to have URL::join function to construct a new URL from an existing URL and a relative path (for example one from Location server header on 302-redirect).

Despite simplicity, implementation has some corner cases, including security sensitive ones, so it is better to have it implemented in a library than to left is an an exercise to a library user.

kvtb avatar Aug 04 '22 23:08 kvtb

Happy to take a PR if you are feeling ambitious

afrind avatar Nov 02 '23 17:11 afrind

Would still accept a PR for this but otherwise it's nonurgent.

afrind avatar Sep 04 '24 20:09 afrind