Skip to content

Add "-i" to sendmail's pipe - #39606

Closed
FedericoHeichou wants to merge 1 commit into
nextcloud:masterfrom
FedericoHeichou:fix-double-dots-with-pipe-sendmail
Closed

Add "-i" to sendmail's pipe#39606
FedericoHeichou wants to merge 1 commit into
nextcloud:masterfrom
FedericoHeichou:fix-double-dots-with-pipe-sendmail

Conversation

@FedericoHeichou

Copy link
Copy Markdown
Contributor

Summary

Symfony Mailer replaces "\n." with "\n..".
If the email to be sent contains links when they are breaklined a line may start with a ., thus can generating a URL like https://www..example.com/[...] or https://www.example.com/index..php/[...] .
With "-i" this replacement is not performed.

TODO

  • ...

Checklist

Signed-off-by: FedericoHeichou <federicoheichou@gmail.com>
@szaimenszaimen added this to the Nextcloud 28 milestone Jul 28, 2023
@szaimenszaimen added the 3. to review Waiting for reviews label Jul 28, 2023
@szaimen
szaimen requested review from a team, ArtificialOwl, icewind1991 and nfebe and removed request for a teamJuly 28, 2023 07:29
@FedericoHeichou
FedericoHeichou deleted the fix-double-dots-with-pipe-sendmail branch July 28, 2023 07:30
@FedericoHeichou

Copy link
Copy Markdown
ContributorAuthor

Sorry, I created a new one. The Github's code is a mess and used the wrong email to perform the commit...
#39607

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Double dots in links in emails

2 participants

@FedericoHeichou@szaimen