[Mimedefang] md_graphdefang_log() must be called "in message context"?

[David F. Skoll]

Gary Funck wrote:

> This check seems overly restrictive.  I notice that
> that md_graphdefang_log() wants to record $Subject and
> $MsgID into log entry, so probably filter_relay() is out,
> but in filter_recipient(), I already have executed
> read_commands_file() before calling md_graphdefang_log(),
> so I presume that both variables are valid?

No.  Neither $Subject nor $MsgID is available until after DATA.
(Think about it...)

> And, a call from filter_relay() would be OK, if the code just
> checked to see where $MsgID and $Subject are defined, and if not,
> md_graphdefang_log() could use either a null string or a question
> mark for their value.

Sure, that's fine with me.  I didn't write (and don't use) graphdefang,
so I didn't write that function.  I'll certainly consider a patch if
anyone wants to submit one...

Regards,

David.