Avoid redundant URL encoding

[zauguin]

The redirect URL for SSO logins is transmitted using a HTML form field, therefore the browser takes care of proper URL encoding. We should therefore not encode the value because otherwise the redirect URL is "double-encoded" and no longer usable.

This fixes a regression introduced by #4220.

Pull Request Checklist

• Pull request is based on the develop branch
• Pull request includes a changelog file
• Pull request includes a sign off

[codecov-io]

Codecov Report

Merging #4555 into develop will increase coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #4555      +/-   ##
===========================================
+ Coverage     74.9%   74.91%   +<.01%     
===========================================
  Files          338      338              
  Lines        34509    34509              
  Branches      5620     5620              
===========================================
+ Hits         25850    25852       +2     
  Misses        7075     7075              
+ Partials      1584     1582       -2

[erikjohnston]

@richvdh does this look plausible, given you touched it last?

[richvdh]

yes, yes it does.

[richvdh]

thanks!