You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by co...@apache.org on 2012/03/16 14:38:07 UTC
svn commit: r1301504 - in /httpd/httpd/trunk: CHANGES server/request.c
Author: covener
Date: Fri Mar 16 13:38:07 2012
New Revision: 1301504
URL: http://svn.apache.org/viewvc?rev=1301504&view=rev
Log:
add filesystem paths to some common error messages.
(slightly modified from dev@ post to not bury the "failed/denied")
Modified:
httpd/httpd/trunk/CHANGES
httpd/httpd/trunk/server/request.c
Modified: httpd/httpd/trunk/CHANGES
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/CHANGES?rev=1301504&r1=1301503&r2=1301504&view=diff
==============================================================================
--- httpd/httpd/trunk/CHANGES [utf-8] (original)
+++ httpd/httpd/trunk/CHANGES [utf-8] Fri Mar 16 13:38:07 2012
@@ -1,6 +1,9 @@
-*- coding: utf-8 -*-
Changes with Apache 2.5.0
+ *) core: Add filesystem paths to access denied / access failed messages
+ AH00035 and AH00036. [Eric Covener]
+
*) core: Add the port number to the vhost's name in the scoreboard.
[Stefan Fritsch]
Modified: httpd/httpd/trunk/server/request.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/request.c?rev=1301504&r1=1301503&r2=1301504&view=diff
==============================================================================
--- httpd/httpd/trunk/server/request.c (original)
+++ httpd/httpd/trunk/server/request.c Fri Mar 16 13:38:07 2012
@@ -1094,9 +1094,9 @@ AP_DECLARE(int) ap_directory_walk(reques
}
else if (APR_STATUS_IS_EACCES(rv)) {
ap_log_rerror(APLOG_MARK, APLOG_ERR, rv, r, APLOGNO(00035)
- "access to %s denied because search "
- "permissions are missing on a component "
- "of the path", r->uri);
+ "access to %s denied (filesystem path '%s') "
+ "because search permissions are missing on a "
+ "component of the path", r->uri, r->filename);
return r->status = HTTP_FORBIDDEN;
}
else if ((rv != APR_SUCCESS && rv != APR_INCOMPLETE)
@@ -1105,7 +1105,8 @@ AP_DECLARE(int) ap_directory_walk(reques
* rather than assume not found.
*/
ap_log_rerror(APLOG_MARK, APLOG_ERR, rv, r, APLOGNO(00036)
- "access to %s failed", r->uri);
+ "access to %s failed (filesystem path '%s')",
+ r->uri, r->filename);
return r->status = HTTP_FORBIDDEN;
}