php mail bug in swift

[guenterpl]

Hi,
I think there is an error in php mail /swift

it call the php mail function with the sender exem. office@xxx.xom

-foffice@xxx.com
instead of statt
-f office@xxx.com

so some mail programm doesnot accept the mail with not valid sender

there is a problem which the $extraparams in swift /transport

best regards
Günter

[MorrisJobke]

cc @ChristophWurst @rullzer

[pawel-slowik]

I don't think this is a bug, at least not in the Nextcloud server.
The Nextcloud server delegates From: address handling to Swift Mailer:

server/lib/private/Mail/Message.php

Lines 40 to 41 in 528eb1b

	/** @var Swift_Message */
	private $swiftMessage;

server/lib/private/Mail/Message.php

Lines 98 to 103 in 528eb1b

	public function setFrom(array $addresses): IMessage {
	$addresses = $this->convertAddresses($addresses);
	$this->swiftMessage->setFrom($addresses);
	return $this;
	}

I'm guessing that this:

there is a problem which the $extraparams in swift /transport

refers to Swift Mailer source code:
https://github.com/swiftmailer/swiftmailer/blob/181b89f18a90f8925ef805f950d47a7190e9b950/lib/classes/Swift/Transport/SendmailTransport.php#L115-L117
https://github.com/swiftmailer/swiftmailer/blob/181b89f18a90f8925ef805f950d47a7190e9b950/lib/classes/Swift/Transport/MailTransport.php#L28-L29

If there's a problem with that, it should be reported to Swift Mailer.

Most likely, hovewer, there is no bug in Swift Mailer either, because the syntax reported as an error is actually correct.
I.e. this is correct: -foffice@xxx.com.
There should be no space between the -f flag and the email address.
Look it up in man sendmail or on https://linux.die.net/man/8/sendmail.sendmail

I suspect that this is a server configuration issue.

[kesselb]

Is this still relevant?

[ghost]

This issue has been automatically marked as stale because it has not had recent activity and seems to be missing some essential information. It will be closed if no further activity occurs. Thank you for your contributions.