FreeRADIUS icon indicating copy to clipboard operation
FreeRADIUS copied to clipboard

Remove URI encoding of user and pass params

Open cwallacetlc opened this issue 3 years ago • 2 comments

The URI encoding is already done within LWP::UserAgent. If you had a space in your password, it would get URI encoded into %20, which would then get encoded again into %2520. It's probably not a big deal, because no one uses spaces in their passwords.... right? ;-)

cwallacetlc avatar Oct 13 '21 16:10 cwallacetlc

Actually i could not reproduce this. Even though spaces in passwords are less common, other non-ASCII characters are used frequently in user names or passwords and thus we need the URI encoding. If the space gets encoded twice in You case, can You add the output of that with debugging enabled in the rlm_perl.ini and using freeradius -X?

plettich avatar Nov 12 '21 14:11 plettich

The URI encoding is already done within LWP::UserAgent.

cwallacetlc avatar Nov 12 '21 17:11 cwallacetlc