[Bug debuginfod/28034] client-side %-escape url characters
Thu Sep 16 10:35:23 GMT 2021
On Mon, 2021-09-13 at 14:11 -0400, Noah Sanci via Elfutils-devel wrote:
> Quick arithmetic change to the original patch with an updated commit
Thanks for commenting this, it is easy to make an off-by-one (or two)
> Previously, urls containing '/', so most urls, would escape '/' to %2F,
> which is undesirable for use in other libraries which may escape
> differently. This patch escapes the '/' and replaces all of them
> ensuring there are no %2Fs sent.
> Some inefficiencies within the code were fixed, such as changing constant
> operations of a while loop within a for loop to a while loop outside of
> a for loop. Also strlen is no longer used within the loop, simplifying
> the interior operations to mere arithmetic.
Could you push it to the master branch?
(Please do rebase first, so we keep a linear history)
More information about the Elfutils-devel