Re: Review Request 60794: Add Cross Origin [CORS] headers for Atlas

2017-07-12 Thread Nixon Rodrigues

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


Ship it!




Ship It!

- Nixon Rodrigues


On July 12, 2017, 8:48 a.m., Ruchi Solani wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60794/
> ---
> 
> (Updated July 12, 2017, 8:48 a.m.)
> 
> 
> Review request for atlas and Nixon Rodrigues.
> 
> 
> Bugs: ATLAS-1896
> https://issues.apache.org/jira/browse/ATLAS-1896
> 
> 
> Repository: atlas
> 
> 
> Description
> ---
> 
> Add Cross Origin [CORS] headers for Atlas, make it configurable from 
> atlas-application properties.
> 
> Properties for headers.
> 
> atlas.headers.Access-Control-Allow-Origin=*
> atlas.headers.Access-Control-Allow-Methods=GET,OPTIONS,HEAD,PUT,POST
> 
> 
> Diffs
> -
> 
>   distro/src/conf/atlas-application.properties 474f253 
>   
> webapp/src/main/java/org/apache/atlas/web/filters/AtlasAuthenticationFilter.java
>  25aede3 
> 
> 
> Diff: https://reviews.apache.org/r/60794/diff/1/
> 
> 
> Testing
> ---
> 
> Adding header properties and tested by API, All Unit test cases are tested.
> 
> 
> Thanks,
> 
> Ruchi Solani
> 
>



Review Request 60794: Add Cross Origin [CORS] headers for Atlas

2017-07-12 Thread Ruchi Solani

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

Review request for atlas and Nixon Rodrigues.


Bugs: ATLAS-1896
https://issues.apache.org/jira/browse/ATLAS-1896


Repository: atlas


Description
---

Add Cross Origin [CORS] headers for Atlas, make it configurable from 
atlas-application properties.

Properties for headers.

atlas.headers.Access-Control-Allow-Origin=*
atlas.headers.Access-Control-Allow-Methods=GET,OPTIONS,HEAD,PUT,POST


Diffs
-

  distro/src/conf/atlas-application.properties 474f253 
  
webapp/src/main/java/org/apache/atlas/web/filters/AtlasAuthenticationFilter.java
 25aede3 


Diff: https://reviews.apache.org/r/60794/diff/1/


Testing
---

Adding header properties and tested by API, All Unit test cases are tested.


Thanks,

Ruchi Solani