Linux Netfilter discussions
 help / color / mirror / Atom feed
* [ulogd2] Small fix for table ulog2 structure for postgresql and two other questions
@ 2012-01-29 19:35 Marcin Mirosław
  0 siblings, 0 replies; only message in thread
From: Marcin Mirosław @ 2012-01-29 19:35 UTC (permalink / raw)
  To: netfilter

Hello,
1. There is a small issue when we create databse structure in 
postgresql. Table "ulog2" has column "timestamp" with default value 
"now". This is incorrect. It means all tuples in this table will have 
the same value, "now" of time when table was created. I guess it isn't 
usefull. This small patch resolves this issue.

--- a/doc/pgsql-ulogd2.sql
+++ b/doc/pgsql-ulogd2.sql
@@ -61,7 +61,7 @@ CREATE TABLE ulog2 (
    ip_fragoff smallint default NULL,
    label smallint default NULL,
    mac_id bigint default NULL,
-  timestamp timestamp NOT NULL default 'now'
+  timestamp timestamp NOT NULL default now()
  ) WITH (OIDS=FALSE);

  CREATE INDEX ulog2_oob_family ON ulog2(oob_family);


2. Ulogd2 obtains list of columns from table defined in config. And 
fails if can't match column name with any name of output key in used 
plugins. Also ignores first column from table which isn't mentioned in 
doc. So when we want to have custom columns in table we have to create 
additional database object "view" to provide proper information for 
ulogd2. Meseems it can be done in a bit cleaner way: let columns with 
first char of name "_" (underscore) will be ignored by stack. In this 
case "view" wouldn't be need to be used.

3. File README contains information about homepage and mailinglist. It 
seems outdated.

(I hope this malinglist is correct for ulogd2)
Regards,
Marcin

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2012-01-29 19:35 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-29 19:35 [ulogd2] Small fix for table ulog2 structure for postgresql and two other questions Marcin Mirosław

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox