[Yanel-commits] rev 60684 - public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet
michi at wyona.com
michi at wyona.com
Tue Sep 13 17:00:55 CEST 2011
Author: michi
Date: 2011-09-13 17:00:54 +0200 (Tue, 13 Sep 2011)
New Revision: 60684
Modified:
public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java
Log:
check whether access logging is applicable refactoring started
Modified: public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java
===================================================================
--- public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java 2011-09-13 15:00:14 UTC (rev 60683)
+++ public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java 2011-09-13 15:00:54 UTC (rev 60684)
@@ -2039,6 +2039,7 @@
// INFO: Check if viewable resource has already created a response
if (!view.isResponse()) {
+ // TODO: Replace with logAccessIsApplicable(view.getMimeType())
if(logAccessEnabled) {
if (view.getMimeType() != null) {
if (isMimeTypeOk(view.getMimeType())) {
@@ -2924,4 +2925,19 @@
return viewId;
}
+
+ /**
+ * Check whether access logging makes sense
+ * @param mimeType Content type of requested resource
+ */
+ private boolean logAccessIsApplicable(String mimeType) {
+ if(logAccessEnabled) {
+ if (mimeType != null) {
+ if (isMimeTypeOk(mimeType)) {
+ return true;
+ }
+ }
+ }
+ return false;
+ }
}
More information about the Yanel-commits
mailing list