Merge pull request #218 from hadfl/issue

fenix: limit the maximum number of digits for issue patterns

Author: oetiker

lib/Fenix/Model/Handler/Issue/Gerrit.pm

@@ -20,7 +20,7 @@ sub issues($self, $msg) {
     my $urlre   = qr!\b\Q$baseurl\E/c/illumos-gate/\+/(\d+)\b!;
     for ($msg) {
         return ([ /$urlre/g ], { url => 1 }) if /$urlre/;
-        return [ /\b(\d{2,})\b/g ] if /\bcode\b/i;
+        return [ /\b(\d{2,5})\b/g ] if /\bcode\b/i;
     }
 
     return [];

lib/Fenix/Model/Handler/Issue/IPD.pm

@@ -19,7 +19,7 @@ sub issues($self, $msg) {
     my $urlre   = qr!\b\Q$baseurl\E/illumos/ipd/\S+/ipd/0+(\d+)/README\.md\b!i;
     for ($msg) {
         return ([ /$urlre/g ], { url => 1 }) if /$urlre/;
-        return [ /\bIPD[-\s]*(\d+)\b/ig ];
+        return [ /\bIPD[-\s]*(\d{1,3})\b/ig ];
     }
 
     return [];

lib/Fenix/Model/Handler/Issue/Illumos.pm

@@ -19,8 +19,8 @@ sub issues($self, $msg) {
     my $urlre   = qr!\b\Q$baseurl\E/issues/(\d+)\b!;
     for ($msg) {
         return ([ /$urlre/g ], { url => 1 }) if /$urlre/;
-        return [ /\b(\d{3,})\b/g ] if /\b(?:illumos|issue)\b/i;
-        return [ /#(\d{3,})\b/g ];
+        return [ /\b(\d{3,5})\b/g ] if /\b(?:illumos|issue)\b/i;
+        return [ /#(\d{3,5})\b/g ];
     }
 
     return [];

lib/Fenix/Model/Handler/Issue/SmartOS.pm

@@ -15,7 +15,7 @@ sub issues($self, $msg) {
     my $urlre   = qr!\b\Q$baseurl\E/bugview/([A-Z]+-\d+)\b!;
     for ($msg) {
         return ([ /$urlre/g ], { url => 1 }) if /$urlre/;
-        return [ /\b([A-Z]+-\d+)\b/g ];
+        return [ /\b([A-Z]+-\d{1,5})\b/g ];
     }
 
     return [];