You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@fineract.apache.org by vi...@apache.org on 2019/05/09 20:08:41 UTC

[fineract] branch develop updated: Revert "Adding a compress response filter, registering filter"

This is an automated email from the ASF dual-hosted git repository.

vishwasbabu pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/fineract.git


The following commit(s) were added to refs/heads/develop by this push:
     new 7bf648a  Revert "Adding a compress response filter,registering filter"
7bf648a is described below

commit 7bf648aa96cc569465d4e379c5af2ed3258ed6db
Author: Vishwas Babu A J <vi...@confluxtechnologies.com>
AuthorDate: Thu May 9 12:57:40 2019 -0700

    Revert "Adding a compress response filter,registering filter"
    
    This reverts commit eb5ad448fa6cb365dad7e2691f4c6f7e9bb1c196.
---
 .../core/boot/WebXmlConfiguration.java             |  3 --
 .../core/filters/ResponseCompressFilter.java       | 43 ---------------------
 .../core/writer/GZipResponseWriter.java            | 45 ----------------------
 3 files changed, 91 deletions(-)

diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/boot/WebXmlConfiguration.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/boot/WebXmlConfiguration.java
index ce2b7e3..6a8f702 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/boot/WebXmlConfiguration.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/boot/WebXmlConfiguration.java
@@ -21,7 +21,6 @@ package org.apache.fineract.infrastructure.core.boot;
 import javax.servlet.Filter;
 import javax.servlet.Servlet;
 
-import org.apache.fineract.infrastructure.core.filters.ResponseCompressFilter;
 import org.apache.fineract.infrastructure.core.filters.ResponseCorsFilter;
 import org.apache.fineract.infrastructure.security.filter.TenantAwareBasicAuthenticationFilter;
 import org.springframework.beans.factory.annotation.Autowired;
@@ -64,8 +63,6 @@ public class WebXmlConfiguration {
         jerseyServletRegistration.addInitParameter("com.sun.jersey.spi.container.ContainerResponseFilters",
                 ResponseCorsFilter.class.getName());
         jerseyServletRegistration.addInitParameter("com.sun.jersey.config.feature.DisableWADL", "true");
-        jerseyServletRegistration.addInitParameter("com.sun.jersey.spi.container.ContainerResponseFilters",
-                ResponseCompressFilter.class.getName());
         // debugging for development:
         // jerseyServletRegistration.addInitParameter("com.sun.jersey.spi.container.ContainerRequestFilters",
         // LoggingFilter.class.getName());
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/filters/ResponseCompressFilter.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/filters/ResponseCompressFilter.java
deleted file mode 100644
index 67185bf..0000000
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/filters/ResponseCompressFilter.java
+++ /dev/null
@@ -1,43 +0,0 @@
-/**
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements. See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership. The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied. See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-package org.apache.fineract.infrastructure.core.filters;
-
-import com.sun.jersey.spi.container.ContainerRequest;
-import com.sun.jersey.spi.container.ContainerResponse;
-import com.sun.jersey.spi.container.ContainerResponseFilter;
-import org.apache.fineract.infrastructure.core.writer.GZipResponseWriter;
-
-import javax.ws.rs.core.HttpHeaders;
-
-/**
- * Filter that compresses the response using gzip
- */
-
-public class ResponseCompressFilter implements ContainerResponseFilter {
-
-    @Override
-    public ContainerResponse filter(ContainerRequest request, ContainerResponse response) {
-        if (request.getRequestHeaders().getFirst(HttpHeaders.ACCEPT_ENCODING).contains("gzip")) {
-            response.getHttpHeaders().add(HttpHeaders.CONTENT_ENCODING, "gzip");
-            response.setContainerResponseWriter( new GZipResponseWriter(response.getContainerResponseWriter()));
-        }
-
-        return response;
-    }
-}
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/writer/GZipResponseWriter.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/writer/GZipResponseWriter.java
deleted file mode 100644
index 1368798..0000000
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/writer/GZipResponseWriter.java
+++ /dev/null
@@ -1,45 +0,0 @@
-/**
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements. See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership. The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied. See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-package org.apache.fineract.infrastructure.core.writer;
-
-import com.sun.jersey.spi.container.ContainerResponse;
-import com.sun.jersey.spi.container.ContainerResponseWriter;
-import java.io.IOException;
-import java.io.OutputStream;
-import java.util.zip.GZIPOutputStream;
-
-public class GZipResponseWriter implements ContainerResponseWriter {
-    private final ContainerResponseWriter crw;
-
-    private GZIPOutputStream gos;
-
-    public GZipResponseWriter(ContainerResponseWriter crw) {
-        this.crw = crw;
-    }
-
-    public OutputStream writeStatusAndHeaders(long contentLength, ContainerResponse response) throws IOException {
-        gos = new GZIPOutputStream(crw.writeStatusAndHeaders(-1, response));
-        return gos;
-    }
-
-    public void finish() throws IOException {
-        gos.finish();
-        crw.finish();
-    }
-}