277106 - Bugzilla::BugMail::MessageToMTA function doesn't have a way to specify header values

Status: RESOLVED DUPLICATE of bug 49893

(Bugzilla :: Email Notifications, defect)

Description

[Andrei Benea]

If we are ever going to solve bug 49893 we need at least the values of "To" and 
"Subject" passed to this function. I suggest using a hash.

Comment 1

[Max Kanat-Alexander]

I'd look at how Mail::Send takes arguments, as that's what we're likely going to
be using as the backend for this at some point. (Or something like that.)

Actually, I think we'd want another wrapper around MessageToMTA for this sort of
functionality. Something like send_message or something. MessageToMTA should
only take the parameters necessary to actually transfer a message to the MTA.

Comment 2

[Max Kanat-Alexander]

Oh, also, this is actually an enhancement -- it doesn't cause any *problems*
with the code that currently exists, it will just allow us to add another feature.

Comment 3

[Dave Miller [:justdave]]

This function was not designed to do anything other than centralize the existing
sendmail calls, using an API quite similar to what was already done (forcing
sendmail to extract that data out of the email).  There are already multiple
bugs which cover adding this functionality to our mail calls, one of which is
actually bug 49893.  I would suggest that adding this functionality before we
actually have a reason to use it would potentially lead to an incomplete
interface which would need to be rewritten anyway once those land. 

*** This bug has been marked as a duplicate of 49893 ***

*** This bug has been marked as a duplicate of 49893 ***

Comment 4

[Vlad Dascalu]

*** Bug 280911 has been marked as a duplicate of this bug. ***