2ndQuadrant / audit-trigger

Simple, easily customised trigger-based auditing for PostgreSQL (Postgres). See also pgaudit.
Other
659 stars 241 forks source link

Fix quote_ident(regclass) #16

Closed memoz closed 9 years ago

memoz commented 9 years ago

https://github.com/2ndQuadrant/audit-trigger/issues/14

Added type cast to TEXT.

memoz commented 9 years ago

@ringerc Hope this doesn't break anything...This audit feature works quite well by the way. :)

ringerc commented 9 years ago

Thanks. I'm glad to hear that. I wrote it years ago and extracted it from the project it was in because I thought it might be useful to others.

These days I'd probably use json storage, but back then hstore was the only option.

nineinchnick commented 9 years ago

Let me just reference my PR #12 with jsonb as a shameless plug.

ringerc commented 9 years ago

@nineinchnick I can't merge that as-is, since it'll break every existing user. I think it'd make sense to wrap it up as a proper extension - a subdir with its own Makefile, .control file, and extension SQL - instead of just replacing the existing file.

nineinchnick commented 9 years ago

@ringerc i know, i'm just referencing it for other people to check it out, maybe somebody would like to try it. Sorry if it's not appropriate.