Feature #5

Change php sendmail to indicate source domain

Added by René Nieuwburg over 2 years ago. Updated 12 months ago.

Status:Closed Start date:08/17/2009
Priority:Normal Due date:
Assignee:René Nieuwburg % Done:

100%
Category:- Spent time: 2.00 hours
Target version:6.1.6
Affected version: Resolution:Resolved

Description

This feature is present (addSendmail method) in the code but was turned off (return null). I am wondering why and would like to know if there is anything against it to reactivate it?

Associated revisions

Revision 048d1c20
Added by René Nieuwburg about 1 year ago

Feature #5 reactivate sendmail domain identification for Apache, change to existing address.

Revision 3deee5cb
Added by Walter Secco 3 months ago

Merge pull request #5 from shakaran/dev

core-ignore-files

History

#1
Updated by Danny Terweij about 2 years ago

Do you have a patch to enable it again?

#2
Updated by Danny Terweij about 1 year ago

  • Status changed from New to Closed
  • Resolution set to No Feedback

#3
Updated by René Nieuwburg about 1 year ago

I forgot about that. I lived in the illusion that the opensourced code would have it active, but you are right, it remains disabled. We would need to test it before. What about including it in a beta version of 6.1.5?

#4
Updated by René Nieuwburg about 1 year ago

Sendmail documentation says:

 -f sender
              Set  the  envelope sender address. This is the address where delivery problems are sent to. With Postfix versions before 2.1, the Errors-To: message header over‐
              rides the error return address.

I think we would need to add that address as a valid address or instead of emailcop@ simply use postmaster@, which is a default existing email address anyhow.

#5
Updated by Danny Terweij about 1 year ago

  • Status changed from Closed to Reopened
  • Assignee set to René Nieuwburg
  • Target version set to 6.1.6
  • Resolution deleted (No Feedback)

You can enable it in trunk. Most dev's have a running svn version to test/code with.

#6
Updated by René Nieuwburg about 1 year ago

Just removing the return null did not do it for me here. Isn't /script/fixweb supposed to include this? Here it doesn't.

#7
Updated by René Nieuwburg about 1 year ago

  • % Done changed from 0 to 100

patched and merged r461 r462

#8
Updated by René Nieuwburg about 1 year ago

  • Resolution set to Resolved

#9
Updated by Danny Terweij 12 months ago

  • Status changed from Reopened to Closed

Also available in: Atom PDF