]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commitdiff
ensure the name parameter starts with a digit
authorJonathan Boyett <jonathan@failingservers.com>
Mon, 11 Jul 2011 17:18:20 +0000 (10:18 -0700)
committerJonathan Boyett <jonathan@failingservers.com>
Mon, 11 Jul 2011 17:18:20 +0000 (10:18 -0700)
lib/puppet/type/firewall.rb

index 078d2bf27239c064516cd731cc166616cd87cd5d..7dbbe9f2a9b7f75192ce45ceb6b5f8a58b005ecd 100644 (file)
@@ -26,10 +26,14 @@ Puppet::Type.newtype(:firewall) do
 
     # Keep rule names simple
     validate do |value|
-      if value !~ /^[a-zA-Z0-9 \-_]+$/ then
+      if value !~ /[a-zA-Z0-9 \-_]+/
         self.fail "Not a valid rule name. Make sure it contains ASCII " \
           "alphanumeric, spaces, hyphens or underscores."
       end
+
+      if value !~ /^\d+/
+        self.fail 'Not a valid rule name. Rules names must start with a digit'
+      end
     end
   end