> On Oct. 8, 2015, 2:47 p.m., Ying Zheng wrote: > > prism/src/main/java/org/apache/falcon/resource/admin/AdminResource.java, > > line 159 > > <https://reviews.apache.org/r/37297/diff/1/?file=1036331#file1036331line159> > > > > From the functionality, it looks more like a PUT method instead of GET. > > See details: > > > > http://www.restapitutorial.com/lessons/httpmethods.html > > > > The HTTP GET method is used to **read** (or retrieve) a representation > > of a resource. > > > > PUT is most-often utilized for **update** capabilities, PUT-ing to a > > known resource URI with the request body containing the newly-updated > > representation of the original resource.
This just manages the cookie handling - Even GET queries will update cookies - Venkat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37297/#review101963 ----------------------------------------------------------- On Aug. 10, 2015, 7:29 a.m., Balu Vellanki wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37297/ > ----------------------------------------------------------- > > (Updated Aug. 10, 2015, 7:29 a.m.) > > > Review request for Falcon, Ajay Yadava, Sowmya Ramesh, and Venkat Ranganathan. > > > Bugs: FALCON-1316 > https://issues.apache.org/jira/browse/FALCON-1316 > > > Repository: falcon-git > > > Description > ------- > > API to get authenticated user > API to clear hadoop.auth cookie > Add property that says if auth is enabled into admin/version API > Allow non-admin users access above apis. > Unit tests. > > > Diffs > ----- > > > common/src/main/java/org/apache/falcon/security/DefaultAuthorizationProvider.java > 449be80 > prism/src/main/java/org/apache/falcon/resource/admin/AdminResource.java > 50b668c > prism/src/test/java/org/apache/falcon/resource/admin/AdminResourceTest.java > PRE-CREATION > > prism/src/test/java/org/apache/falcon/resource/admin/MockHttpServletResponse.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/37297/diff/ > > > Testing > ------- > > End2End testing done on multiple operating systems. > > > Thanks, > > Balu Vellanki > >
