sender_url should not contain mailto:

#1 is probably a bug. It comes from ContactEmailBuilderBase.php line 30 which adds in the mailto: . I don't really know why😃.

#2 should be corrected by "URL to absolute" in your All/All email policy. Maybe you removed it by mistake?

Thanks very much for the reply. And in light of this, then it looks like there may be 2 bugs ...

For #1, it's good to at least know the likely cause of this. It's obviously not essential, but would be nice someday if this just showed the e-mail address like the default Drupal setup.

For #2, I double checked and 'URL to absolute' is there. I've attached a screenshot from /admin/config/system/mailer, which shows that it's active. As such, I'm guessing that is a bug as well.

Any ideas to troubleshoot this would be much appreciated. And naturally if you can fix either or both of these bugs, that would be fantastic!

#1 if someone can make a merge request that does what I described in #3 then I will commit it

#2 is working fine for me, so it needs steps to reproduce. If you could try on a clean site the problem might go away.

Prem Suthar → made their first commit to this issue’s fork.

Update the Changes Please Review.

Thanks

Wait a minute.

The mailto: is there because the variable is supposed to be a URL, which you can put it inside a href. The default policy contains this line, and this patch has broken it.

<a href="{{ sender_url }}">{{ sender_name }}</a> has sent you a message via your form at {{ site_name }}.

Perhaps instead we could make a new variable sender_mail? However in that case it would change behaviour for a registered user, when the URL will be something like /user/{user}.

Perhaps you can remove the mailto: in twig? See https://stackoverflow.com/questions/15600626/begins-with-in-twig-template.