X-Git-Url: https://juplo.de/gitweb/?p=percentcodec;a=blobdiff_plain;f=cachecontrol%2Fsrc%2Fmain%2Fjava%2Fde%2Fhalbekunst%2Fjuplo%2Fcachecontrol%2FAcceleratorFilter.java;h=e881a8ed82c71e5021b4df290ed98a37d0e7d6bf;hp=f9cd886f6d3c6d5a6fd1a5c58cad33a60c39c589;hb=baf207bba5be3121a57f76f2e70231c01180b34c;hpb=5f8def1103a2a408579d1b6ae29e447a272a9558 diff --git a/cachecontrol/src/main/java/de/halbekunst/juplo/cachecontrol/AcceleratorFilter.java b/cachecontrol/src/main/java/de/halbekunst/juplo/cachecontrol/AcceleratorFilter.java index f9cd886f..e881a8ed 100644 --- a/cachecontrol/src/main/java/de/halbekunst/juplo/cachecontrol/AcceleratorFilter.java +++ b/cachecontrol/src/main/java/de/halbekunst/juplo/cachecontrol/AcceleratorFilter.java @@ -62,21 +62,18 @@ public class AcceleratorFilter implements Filter { return; } - // TODO: Das macht so wahrscheinlich keinen Sinn... - /** Prüfen, ob es sich um eine Anfrage für einen JSP-Include handelt */ - if (request.getAttribute(REQUEST_URI_ATTRIBUTE) != null) { - log.debug("Includes cannot be accelerated"); - chain.doFilter(request, response); - return; - } - HttpServletRequest httpRequest = (HttpServletRequest)request; HttpServletResponse httpResponse = (HttpServletResponse)response; AccelerationWrapper wrapper = new AccelerationWrapper(httpRequest, httpResponse); httpRequest.setAttribute(RESPONSE_WRAPPER, wrapper); cacheControl.init(wrapper); - chain.doFilter(request, wrapper); - wrapper.finish(); + try { + chain.doFilter(request, wrapper); + wrapper.finish(); + } + catch (NotModifiedException nm) { + log.trace("Not modified: {}", httpRequest.getRequestURI()); + } } @Override @@ -344,8 +341,6 @@ public class AcceleratorFilter implements Filter { writer.flush(); else if (stream != null) stream.flush(); - - out.flush(); } @Override @@ -652,8 +647,14 @@ public class AcceleratorFilter implements Filter { out().write(buffer); committed = true; /** Dekoration nur beim ersten Schreib-Schub anstoßen */ - if (pos == size) - cacheControl.decorate(request, response); + if (pos == size) { + if (!cacheControl.decorate(request, response)) { + zipped = false; + os = null; + pos = 0; + throw new NotModifiedException(); + } + } pos = 0; } buffer[pos++] = (byte) i; @@ -667,8 +668,14 @@ public class AcceleratorFilter implements Filter { committed = true; /** Dekoration nur beim ersten Schreib-Schub anstoßen */ - if (pos == size) - cacheControl.decorate(request, response); + if (pos == size) { + if (!cacheControl.decorate(request, response)) { + zipped = false; + os = null; + pos = 0; + throw new NotModifiedException(); + } + } out().write(buffer, 0, pos); out().flush(); pos = 0; @@ -679,7 +686,8 @@ public class AcceleratorFilter implements Filter { if (size == 0) { committed = true; zipped = false; - cacheControl.decorate(request, response); + if (!cacheControl.decorate(request, response)) + throw new NotModifiedException(); sos.close(); } else { @@ -690,3 +698,5 @@ public class AcceleratorFilter implements Filter { } } } + +class NotModifiedException extends IOException {} \ No newline at end of file