Lack of interest Change \XF\EmailBounce\Processor :: logBounceMessage to public

This suggestion has been closed automatically because it did not receive enough votes over an extended period of time. If you wish to see this, please search for an open suggestion and, if you don't find any, post a new one.

DragonByte Tech

Well-known member
I'm writing an integration with a 3rd party mailing list service (SendGrid) and it has its own event / webhook system for letting us know when an email bounced. It doesn't give us the same raw data as the normal bounce handler does, but I would still like to log the bounce so that the administrator is kept aware of the fact that a SendGrid email bounced.

I'm having to copy the entire logBounceMessage function (thus losing any integration with 3rd party mods) because it's currently set to protected.

I realise that this would be a bit of a BC breaking change, so perhaps a compromise would be to create a new function in \XF\EmailBounce\Processor, something like
PHP:
public function logBounce($rawMessage, $action, ParseResult $result, User $user = null)
{
    $this->logBounceMessage($rawMessage, $action, $result, $user);
}

Thanks for considering.
 
Upvote 2
This suggestion has been closed. Votes are no longer accepted.
Back
Top Bottom