Raj Kumar wrote: > > hi dirk, > I saw that you changed the parsePrivilege method of the AclMethod > to only look for all,read,read-acl,write,write-acl and ignore the rest of > the privileges but the parseRequest method > of the AclMethod still has case statements for PRIVILEGE_READ_OBJECT and > PRIVILEGE_READ_LOCKS etc which are redundant > and can be removed . Though the specs does not say anyting about the > status code for a request containing a unsupported privilege i would think > that BAD_REQUEST is the right reponse code > thanks, > rajkumar
Yes, I left those redundant case statements in there because I suspect that in a few weeks/months we are going to enable them. I could have put those statements in comment but there is no major advantage in doing that. Anyway the code is correct, it isn't used, but it's still correct. Thanks, Dirk
