Hi,
For EventTriggerOnConnect():

+           PG_CATCH();
+           {
...
+               AbortCurrentTransaction();
+               return;

Should runlist be freed in the catch block ?

+           gettext_noop("In case of errors in the ON client_connection
EVENT TRIGGER procedure, this parameter can be used to disable trigger
activation and provide access to the database."),

I think the text should be on two lines (current line too long).

Cheers

Reply via email to