[Keycloak] The body of the email-verification was broken.

[monaka]

Describe the bug

The body of the mail from Keycloak was broken.

Che version

• nightly

Steps to reproduce

0. Registrer your account on Keycloak.
1. Read the confirmation mail from Keycloak.
2. You'll got a broken HTML mail.

Expected behavior

A valid HTML mail arrival.

Runtime

• kubernetes (include output of kubectl version)

Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.0", GitCommit:"e8462b5b5dc2584fdcd18e6bcfe9f1e4d970a529", GitTreeState:"clean", BuildDate:"2019-06-21T13:09:06Z", GoVersion:"go1.12.6", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.0", GitCommit:"641856db18352033a0d96dbc99153fa3b27298e5", GitTreeState:"clean", BuildDate:"2019-03-25T15:45:25Z", GoVersion:"go1.12.1", Compiler:"gc", Platform:"linux/amd64"}

Screenshots

Installation method

• I don't know Customized Helm Charts

Environment

• my computer
  • Linux ChromeOS
• Cloud
  • Azure

Additional context

Looks HTML content was escaped by msg() in FTL template.

[monaka]

The similar issue to keycloak/keycloak#4988 ?

[rhopp]

@monaka Do you think this bug is important enought to be prioritized into Che 7 GA release?
cc @slemeur

[monaka]

The viewpoint of functionality, this is not critical. Users who want to registration can click the link (or at least can copy/paste it to their web browser).

IMO, it's not blocker and just better to apply if someone could make patches before Che 7 GA release.

[slemeur]

I think this should be landing in 7.1.0 cc @l0rd

[skabashnyuk]

We can handle that.

[slemeur]

Thank you @skabashnyuk . You can handle it for 7.0?

[rhopp]

I've just had instance running, so I tried this. Indeed adding ?no_esc to few places solves this issue (I've tried that on running instance changing this inside container, thus no PR).

[skabashnyuk]

Thank you @skabashnyuk . You can handle it for 7.0?

I think we can try.