Pablo Neira Ayuso wrote: > On Wed, Aug 01, 2012 at 11:27:16PM +0200, Eric Leblond wrote: >> This patch adds a timestamp option to the nfacct plugin. >> If activated, nfacct output a timestamp which is computed just >> after sending the nfacct request. > > I think it makes sense to make it unconditionally. Yep. The PGSQL output plugin can't function without it. When I have the following: stack=acct1:NFACCT,pgsql3:PGSQL I get the following error if timestamp is not enabled: <7> ulogd.c:700 type mismatch between PGSQL and NFACCT in stack <7> ulogd.c:727 cannot find key `oob.time.sec' in stack I had to dig in further to find out why that was the case (something I could have done without), so I think it makes sense for the timestamp to always be included. -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
