chaitalicod commented on code in PR #482:
URL: https://github.com/apache/atlas/pull/482#discussion_r2633759019


##########
webapp/src/main/java/org/apache/atlas/web/security/AtlasSecurityConfig.java:
##########
@@ -195,7 +195,8 @@ protected void configure(HttpSecurity httpSecurity) throws 
Exception {
         //@formatter:off
         httpSecurity.authorizeRequests().anyRequest().authenticated()
                 .and()
-                .headers()
+                // Why disable() xssProtection -> By default Spring Security 
automatically adds security headers unless you disable them. No Modern Browsers 
support and its replaced by "Content-Security-Policy"

Review Comment:
   Check if this comment is required here or make it precise and relevant



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to