-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69916/#review212638
-----------------------------------------------------------


Ship it!




Ship It!

- Zsombor Gegesy


On Feb. 7, 2019, 3:48 p.m., Andras Salamon wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/69916/
> -----------------------------------------------------------
> 
> (Updated Feb. 7, 2019, 3:48 p.m.)
> 
> 
> Review request for oozie, Denes Bodo, Kinga Marton, and Mate Juhasz.
> 
> 
> Repository: oozie-git
> 
> 
> Description
> -------
> 
> OOZIE-3427 - Use best practices in HTTP response headers
> 
> 
> Diffs
> -----
> 
>   core/src/main/java/org/apache/oozie/servlet/HttpResponseHeaderFilter.java 
> PRE-CREATION 
>   core/src/test/java/org/apache/oozie/servlet/DagServletTestCase.java 
> ce731a16b 
>   
> core/src/test/java/org/apache/oozie/servlet/TestAuthFilterAuthOozieClient.java
>  bc469e260 
>   
> core/src/test/java/org/apache/oozie/servlet/TestBulkMonitorWebServiceAPI.java 
> b4054b05f 
>   
> core/src/test/java/org/apache/oozie/servlet/TestHttpResponseHeaderFilter.java 
> PRE-CREATION 
>   server/src/main/java/org/apache/oozie/server/FilterMapper.java 3dc9be815 
>   server/src/test/java/org/apache/oozie/server/TestEmbeddedOozieServer.java 
> 58543e6fd 
>   webapp/src/main/webapp/WEB-INF/web.xml 2edbdf153 
> 
> 
> Diff: https://reviews.apache.org/r/69916/diff/2/
> 
> 
> Testing
> -------
> 
> Tested embedded jetty and war installed to a local tomcat. Local tomcat was 
> hardly working, but at least I was able to test /versions.
> 
> $ wget -qS http://localhost:11000/oozie/index.jsp
> HTTP/1.1 200 OK
>   Date: Thu, 07 Feb 2019 09:44:32 GMT
>   X-Frame-Options: DENY
>   Content-Type: text/html;charset=utf-8
>   Set-Cookie: JSESSIONID=1lx0y9fy2pd6n1rh911vc2l1sd;Path=/oozie
>   Expires: Thu, 01 Jan 1970 00:00:00 GMT
>   Content-Length: 3739
> 
> 
> Thanks,
> 
> Andras Salamon
> 
>

Reply via email to