[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[SCM] Debian package checker branch, master, updated. 2.5.15-4-g38184ad



The following commit has been merged in the master branch:
commit 38184add716c6d9d24caf0018c13250ed223b4f2
Author: Jean-Michel Vourgère <jmv_deb@nirgal.com>
Date:   Sat Jul 27 22:00:13 2013 +0200

    c/apache.pm: Ignore "AllowOverride" etc.
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/checks/apache2.pm b/checks/apache2.pm
index b7f4041..a1e030b 100644
--- a/checks/apache2.pm
+++ b/checks/apache2.pm
@@ -229,8 +229,9 @@ sub inspect_conf_file {
     open(my $fd, '<', $filename);
     while (<$fd>)  {
 
-        for my $directive ('Order', 'Satisfy', 'Allow', 'Deny', '<(|/)Limit.*?>', '<(|/)LimitExcept.*?>') {
-            if (m/($directive)/) {
+        for my $directive ('Order', 'Satisfy', 'Allow', 'Deny',
+                           qr{</?Limit.*?>}xsm, qr{</?LimitExcept.*?>}xsm) {
+            if (m{\A \s* ($directive) (?:\s+|\Z)}xsm) {
                 tag 'apache2-deprecated-auth-config', $1;
             }
         }
diff --git a/debian/changelog b/debian/changelog
index 4387723..bc74bd3 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,5 +1,11 @@
 lintian (2.5.16) UNRELEASED; urgency=low
 
+  * checks/apache2.pm:
+    + [NT] Fix F-P apache2-deprecated-auth-config for
+      "AllowOverride".  Thanks to Jean-Michel Vourgère
+      for the report and an initial patch.
+      (Closes: #717621)
+
   * frontend/lintian:
     + [NT] Move some handling of overrides to Lintian::Tags.
 
diff --git a/t/tests/apache2-webapplications-general/debian/phpmyfoo.conf b/t/tests/apache2-webapplications-general/debian/phpmyfoo.conf
index 32e42f2..30909fe 100644
--- a/t/tests/apache2-webapplications-general/debian/phpmyfoo.conf
+++ b/t/tests/apache2-webapplications-general/debian/phpmyfoo.conf
@@ -9,5 +9,6 @@ Alias /foo /usr/share/foo
 	  Order deny, allow
 	  Require valid-user
   </Limit>
+  AllowOverride None
 </Directory>
 

-- 
Debian package checker


Reply to: