fix: use html/template for SAML POST/Logout templates

[coelho]

Follow up to #29 - Looks like the previous patch was incomplete.

Currently text/template is used, which does not properly HTML Encode the HTML Form values used in SAML POST/Logout flows.
Switching to html/template resolves this issue.

[codecov]

Codecov Report

Merging #32 (0c5648c) into main (80e4820) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #32   +/-   ##
=======================================
  Coverage   47.38%   47.38%           
=======================================
  Files          21       21           
  Lines        1967     1967           
=======================================
  Hits          932      932           
  Misses        977      977           
  Partials       58       58           

Impacted Files	Coverage Δ
pkg/provider/identityprovider.go	56.06% <ø> (ø)
pkg/provider/logout_response.go	0.00% <ø> (ø)
pkg/provider/response.go	61.34% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[coelho]

Added 0c5648c

Confirmed with these patches:

• Login is working perfect with Google third-party SAML login.
• Logout still seems buggy (infinite redirect loop), but this is not a regression. It does not work in prior versions either.

Let me know if you have any questions.

[mffap]

@stebenz could you take this over?
@hifabienne would it make sense to extend the action and also assign PRs to the kanban board?

[hifabienne]

@stebenz could you take this over? @hifabienne would it make sense to extend the action and also assign PRs to the kanban board?

Not sure, because most of the time a pr is linked to an issue, if we have both on the board it is kind of duplicate

[github-actions]

🎉 This PR is included in version 0.0.9 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀