Re: cvs: php4 /ext/standard mail.c php_mail.h

On Mon, 5 Feb 2001, Sterling Hughes wrote: > > @- Added a new parameter to mail() which appends aditional command line > > @ parameters to the mail program. (Derick) > > This won't be portable, on windows it uses raw smtp code... I know this... it should probably be noted in the docs. > Are you sure we should be adding that option to the mail() command? If the > user really needs the advanced features provided by sendmail, they can > easily use popen() to achieve this... I think the -f option to sendmail is not advanced, and that because that without this -f option, all mail seems to come from nobody@localhost, this clutters up the logs, and no stats can be gegenerated about the usage of the scripts that send mail. The popen is possible too of course, but I think it's not that easy to do for almost not experienced users. To lower the bar for these users (and for making the transistion easier) I think this functionality is justified to be included in the CVS tree. For security, I thought about makeing the fifth parameter only be used for the -f option only, but this would be troublesome for other mailers probably. However, I really think that restricting it to the -f option is better. Derick Rethans --------------------------------------------------------------------- PHP: Scripting the Web - www.php.net - derick@php.net --------------------------------------------------------------------- JDI Media Solutions - www.jdimedia.nl - d.rethans@jdimedia.nl H.v. Tussenbroekstraat 1 - 6952 BL Dieren - The Netherlands ---------------------------------------------------------------------