Tom Eastep wrote: > On 04/25/2013 08:30 AM, Dash Four wrote: > >> Dash Four wrote: >> >>> The only thing I could suggest is what you've already done - altering >>> the email, but as I already pointed out above - I am not really that >>> bothered if I am shown to be "the author" or not. >>> >> I hope it is not too late to request one minor correction, as far as >> nfacct object name matching is concerned (that will affect all NFACCT >> statements): Currently shorewall only matches/accepts a "word" character >> (i.e. [a-zA-Z_0-9]), but I would like to use other characters, like "%", >> "&", "@" and "~" (the latter two being particularly important as this is >> how I split my traffic into "sub-classes"). Would that be possible? >> > > Please give the attached patch a try. It also corrects a problem where > not all nfacct objects were created by the generated script. > Thanks, Tom - will give it a go when I get home in a few hours.
I had a quiet chuckle at all the Chains.pm changes you've made - I did look at that code before and was scratching my head how on earth is this going to insert the nfacct names into nfobjects (the variable you use to store all nfacct names), but since my perl skills were insufficient (I thought the "push" function "somehow did it") I let it all go. ------------------------------------------------------------------------------ Try New Relic Now & We'll Send You this Cool Shirt New Relic is the only SaaS-based application performance monitoring service that delivers powerful full stack analytics. Optimize and monitor your browser, app, & servers with just a few lines of code. Try New Relic and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_apr _______________________________________________ Shorewall-devel mailing list Shorewall-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/shorewall-devel
