Escape provider name in oauth2 provider redirect

[zeripath]

Fix #12647

Signed-off-by: Andrew Thornton [email protected]

[6543]

will need a backport

[codecov-commenter]

Codecov Report

Merging #12648 into master will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #12648      +/-   ##
==========================================
- Coverage   43.33%   43.32%   -0.01%     
==========================================
  Files         646      646              
  Lines       71588    71588              
==========================================
- Hits        31020    31016       -4     
- Misses      35558    35561       +3     
- Partials     5010     5011       +1     

Impacted Files	Coverage Δ
modules/auth/oauth2/oauth2.go	27.58% <0.00%> (ø)
modules/util/timer.go	42.85% <0.00%> (-42.86%)	⬇️
modules/util/remove.go	65.38% <0.00%> (-15.39%)	⬇️
services/pull/update.go	51.92% <0.00%> (-5.77%)	⬇️
services/pull/temp_repo.go	26.59% <0.00%> (-3.20%)	⬇️
services/pull/check.go	47.69% <0.00%> (-2.31%)	⬇️
modules/log/event.go	57.54% <0.00%> (-1.89%)	⬇️
modules/log/file.go	73.60% <0.00%> (-1.61%)	⬇️
services/pull/pull.go	42.03% <0.00%> (-0.70%)	⬇️
models/error.go	34.81% <0.00%> (-0.52%)	⬇️
... and 8 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9bc69ff...c1a7157. Read the comment docs.

[6543]

backport -> #12650

[lafriks]

🚀