Re: assertion failed: imap/mboxevent.c: 743: filled_params(type, event)

HI Ellie,

I agree that it's probably a bug.  I'll open a github issue.

I'll report back with the issue number.

Thanks,
Matt

On 7/16/20 5:47 PM, ellie timoney wrote:
> Hi,
>
> I've seen something like this before, and my gut feel is that this is going to turn out to be a bug in Cyrus.
>
> I think what's happening is that, somewhere in Cyrus, an event is being generated with a type that's supposed to contain a serverAddress field, but the serverAddress field is not being initialised.
>
> Before a generated event actually gets sent out to the notifier, we validate that all the required parameters have been filled ("filled_params()"), and the fatal assertion is telling us that this one has not been, even though it should have been.
>
> Would you like to open a GitHub issue at https://github.com/cyrusimap/cyrus-imapd/issues ?  If you don't, I will.  But if I do it, then you won't get automatic notifications about updates, so if you can, it's better if you do it.  Feel free to just paste your previous email as the issue text. :)
>
> Cheers,
>
> ellie
>
> On Thu, Jul 16, 2020, at 11:23 AM, Matthew Schumacher wrote:
>> I'm trying to use external notifications on 3.2.2 but it doesn't work.
>> If I define
>>
>> event_notifier: external
>> notify_external: /usr/cyrus/bin/cyrus_notify
>> event_groups: access
>> event_extra_params: clientAddress timestamp service
>>
>> Then the imapd thread dies with this assertion:
>>
>> Jul 15 18:01:54 snow imaps[28108]: Cannot notify event Login: missing
>> parameters: serverAddress clientAddress
>> Jul 15 18:01:54 snow imaps[28108]: Fatal error: Internal error:
>> assertion failed: imap/mboxevent.c: 743: filled_params(type, event)
>>
>> If I remove event_groups and event_extra_params then notify never calls
>> my external script and notify breaks.
>>
>> If I define "event_groups: access" and omit event_extra_params then I'm
>> back to:
>>
>> Jul 15 18:14:26 snow imaps[28934]: Cannot notify event Login: missing
>> parameters: serverAddress
>> Jul 15 18:14:26 snow imaps[28934]: Fatal error: Internal error:
>> assertion failed: imap/mboxevent.c: 743: filled_params(type, event)
>>
>> Anyone know where this serverAddress is coming from and how to fix it?
>>
>> schu
>> ----
>> Cyrus Home Page: http://www.cyrusimap.org/
>> List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/
>> To Unsubscribe:
>> https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus
> ----
> Cyrus Home Page: http://www.cyrusimap.org/
> List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/
> To Unsubscribe:
> https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus

----
Cyrus Home Page: http://www.cyrusimap.org/
List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/
To Unsubscribe:
https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus