Dan Carley

  • Registered on: 10/01/2009
  • Last connection: 12/07/2011

Projects

Activity

Reported issues: 14

11/30/2011

03:36 am Puppet Labs Modules Feature #11091 (Duplicate): Add a position attribute to firewall rules.
Hi Steve,
This is covered by #10275. Linking to that.
03:34 am Puppet Labs Modules Feature #10275: firewall: proposal - 'order' parameter
I have a branch on the go for this: https://github.com/dcarley/puppetlabs-firewall/tree/10275-order_param
It now a...

11/24/2011

08:10 am Puppet Labs Modules Bug #11034 (Merged - Pending Release): firewall: Whitespace purge
Reviewed with `git diff -b`. CLA is covered in #10997. Merged.

11/14/2011

03:38 am Puppet Labs Modules Bug #10239: Unable to purge rules not matching regex
Discarding the pull request in favour of #10700 (which relaxes alphanumeric checks) and #10275 (which should remove t...

10/25/2011

02:24 pm Puppet Labs Modules Bug #10239: Unable to purge rules not matching regex
Yeah, length is the only limitation on iptable comments. Any characters are allowed.
http://git.netfilter.org/cgi-...
12:42 pm Puppet Labs Modules Bug #10239: Unable to purge rules not matching regex
Sure. I've got a patch at work which also reduces the name regex to `/^\d/` to satisfy the variety of different comme...

10/24/2011

09:02 am Puppet Labs Modules Bug #10239: Unable to purge rules not matching regex
It's not just number prefixes. I don't think there's any restriction on the characters that can go into a comment str...
07:37 am Puppet Labs Modules Bug #10239: Unable to purge rules not matching regex
I'm inclined to agree. It's a kind of arbitrary restriction. However I almost prefer it to the workarounds I've had t...
03:48 am Puppet Labs Modules Bug #10239: Unable to purge rules not matching regex
Pull request submitted.
https://github.com/puppetlabs/puppetlabs-firewall/pull/19
Arguably "`! hash[:name] or`"...
03:41 am Puppet Labs Modules Bug #10239 (Code Insufficient): Unable to purge rules not matching regex
The puppetlabs-firewall module is unable to purge existing IPtables rules that don't have a comment matching the rege...

Also available in: Atom