Opened on 01/14/2015 at 01:21:55 PM

Closed on 01/14/2015 at 02:21:16 PM

Last modified on 02/04/2015 at 07:09:21 PM

#1803 closed change (fixed)

Do not use random email addresses in From: ( contact form)

Reported by: trev Assignee: matze
Priority: P3 Milestone:
Module: Websites Keywords:
Cc: arthur, till Blocked By:
Blocking: #1753 Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working:
Review URL(s):


Background contact form will send a mail according to the sitescripts/formmail/template/eyeo.mail template. This template currently puts the supplied sender's address into From: field of the email. This is problematic as we don't own the sender's domain and might be violating SPF policies.

What to change

Use From: and put the sender's address under Reply-To:.

Attachments (0)

Change History (7)

comment:1 Changed on 01/14/2015 at 01:30:39 PM by matze

  • Owner set to matze

comment:2 Changed on 01/14/2015 at 01:32:54 PM by matze

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:3 Changed on 01/14/2015 at 01:47:42 PM by matze

  • Blocking 1753 added

comment:4 Changed on 01/14/2015 at 02:21:16 PM by matze

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:5 Changed on 01/14/2015 at 02:48:08 PM by saroyanm

comment:6 Changed on 02/04/2015 at 10:17:27 AM by trev

  • Cc arthur till added

Arthur notes that this breaks forwarding emails now - the forwarded email no longer has the address of the original sender. The way I see it, there is little we can do about this. Maybe forwarding in Gmail can be adjusted somehow to use Reply-To address? Otherwise people will have to add the original sender explicitly when forwarding mails.

comment:7 Changed on 02/04/2015 at 07:09:21 PM by matze

We will look into this issue, of course. But in the context of a new ticket - this one here can remain closed (because it's an unpleasant side-effect of a mandatory change, not a bug with the change-set). Thus, please use the new ticket (#1946) for any further discussion. Thank you!

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from matze.
Note: See TracTickets for help on using tickets.