Feature #65791

Using PHP configured sendmail paths, if [MAIL][transport] = sendmail is active

Added by Kevin Häfeli over 4 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Could have
Assignee:
Category:
Miscellaneous
Target version:
Start date:
2015-03-17
Due date:
% Done:

100%

PHP Version:
5.6
Tags:
Complexity:
easy
Sprint Focus:
Remote Sprint

Description

Hi,

TYPO3 is using 'transport_sendmail_command' => '/usr/sbin/sendmail -bs' per default in /typo3/sysext/core/Configuration/DefaultConfiguration.php
"The command to call to send a mail locally. The default works on most modern UNIX based mail server (sendmail, postfix, exim)"

But this is not 100% correct, not every modern sendmail (e.g. msmtp) implementation is supporting the "-bs" option.
Sometimes for example "-t" is needed. OK, you can set it manually. But:

I think most of the hoster out in the web know their environment.
So why not trusting the hoster and using the PHP "sendmail_path" in "transport_sendmail_command" as default?

Cheers,
Kevin

Associated revisions

Revision 637ebea6 (diff)
Added by Romain Leleu about 4 years ago

[FEATURE] Retrieve sendmail command from PHP configuration

The install tool setting [MAIL][transport_sendmail_command] is now retrieved
from PHP configuration sendmail_path during installation.
As before this settings is only used if you set sendmail as [MAIL][transport]

Resolves: #65791
Releases: master
Change-Id: I06bfdcd4c2c3de3e8c340a15c5a5573cfbe6c678
Reviewed-on: http://review.typo3.org/40312
Reviewed-by: Daniel Goerz <>
Tested-by: Daniel Goerz <>
Reviewed-by: Daniel Maier <>
Tested-by: Daniel Maier <>
Reviewed-by: Stefan Neufeind <>
Tested-by: Stefan Neufeind <>

History

#1 Updated by Markus Klein over 4 years ago

  • Target version changed from 7.1 (Cleanup) to 7.2 (Frontend)

Generally I like the idea.
I'd suggest to add this detection to the Install Tool, where the system detects the settings.
Can you provide a patch to our review system for that?

Since this should only affect new installations, it should not touch the setting itself, but only during initial installation.
We can therefore even backport this to 6.2 then.

#2 Updated by Benni Mack over 4 years ago

  • Target version changed from 7.2 (Frontend) to 7.4 (Backend)
  • Sprint Focus set to On Location Sprint

#3 Updated by Romain Leleu over 4 years ago

  • Status changed from New to In Progress
  • Assignee set to Romain Leleu

#4 Updated by Gerrit Code Review over 4 years ago

  • Status changed from In Progress to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40312

#5 Updated by Gerrit Code Review over 4 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40312

#6 Updated by Gerrit Code Review over 4 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40312

#7 Updated by Gerrit Code Review over 4 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40312

#8 Updated by Gerrit Code Review about 4 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40312

#9 Updated by Gerrit Code Review about 4 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40312

#10 Updated by Gerrit Code Review about 4 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40312

#11 Updated by Gerrit Code Review about 4 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40312

#12 Updated by Susanne Moog about 4 years ago

  • Target version changed from 7.4 (Backend) to 7.5

#13 Updated by Gerrit Code Review about 4 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40312

#14 Updated by Gerrit Code Review about 4 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40312

#15 Updated by Anja Leichsenring about 4 years ago

  • Sprint Focus changed from On Location Sprint to Remote Sprint

#16 Updated by Gerrit Code Review about 4 years ago

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40312

#17 Updated by Romain Leleu about 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#18 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF