Re: pgsql: Add pg_audit, an auditing extension

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Stephen Frost <sfrost(at)snowman(dot)net>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Add pg_audit, an auditing extension
Date: 2015-05-14 14:59:01
Message-ID: 20267.1431615541@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Stephen Frost <sfrost(at)snowman(dot)net> writes:
> And... I busted the buildfarm. Will fix.

-- Load pg_audit module
create extension pg_audit;
+ ERROR: pg_audit must be loaded via shared_preload_libraries

This seems like a rather poorly thought-through error check.
It will break not only the buildfarm but any dump/restore scenario.
You really can't have extensions that refuse to let themselves
be created.

regards, tom lane

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Stephen Frost 2015-05-14 15:00:47 Re: pgsql: Add pg_audit, an auditing extension
Previous Message Stephen Frost 2015-05-14 14:57:24 pgsql: Fix buildfarm with regard to pg_audit

Browse pgsql-hackers by date

  From Date Subject
Next Message Stephen Frost 2015-05-14 15:00:47 Re: pgsql: Add pg_audit, an auditing extension
Previous Message Tom Lane 2015-05-14 14:54:34 Re: upper planner path-ification