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