[ 
https://issues.apache.org/jira/browse/HADOOP-13008?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15274571#comment-15274571
 ] 

Chris Nauroth commented on HADOOP-13008:
----------------------------------------

Hello [~lmccay].  There are a few more nitpicky things I forgot in my last 
review:

# Please add the visibility annotations {{@InterfaceAudience.Public}} and 
{{@InterfaceStability.Evolving}} to {{XFrameOptionsFilter}}.
# To satisfy Checkstyle, let's add a basic package-info.java file.  This isn't 
a problem introduced by the current patch, but there are public classes in 
there, so it would be nice to have the package-info.java.

> Add XFS Filter for UIs to Hadoop Common
> ---------------------------------------
>
>                 Key: HADOOP-13008
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13008
>             Project: Hadoop Common
>          Issue Type: New Feature
>          Components: security
>            Reporter: Larry McCay
>            Assignee: Larry McCay
>             Fix For: 2.8.0
>
>         Attachments: HADOOP-13008-001.patch, HADOOP-13008-002.patch
>
>
> Cross Frame Scripting (XFS) prevention for UIs can be provided through a 
> common servlet filter. This filter will set the X-Frame-Options HTTP header 
> to DENY unless configured to another valid setting.
> There are a number of UIs that could just add this to their filters as well 
> as the Yarn webapp proxy which could add it for all it's proxied UIs - if 
> appropriate.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to