Details
-
Bug
-
Status: Closed
-
Major
-
Resolution: Fixed
-
JCR Jackrabbit Access Manager 2.1.0
-
None
Description
as per http://markmail.org/thread/7niaxhzxhxv4a2lo
"
Hi,
I am not certain if this was intentional but the format of GetAclServlet was
changed by [1].
Previously the format was a map keyed by principalid, now its an array
containing objects.
I can understand that an array maintains the order of the ACE's in the ACL which
is required by parts of SLING-1458, however its not backwards compatible and
breaks all existing clients that were expecting a map. Since that might be quite
a lot of code, embedded in Client side Javascript apps and server side http
clients, wouldn't it be better to provide keep the format and add a sequence
number to the map objects? , eg
Old format:
{"user1-1274468817" : {"granted":["jcr:read"]}}
New format
[
]
Suggested new format
{"user1-1274468817" : {"granted":["jcr:read"], "sequence": 0}}
I know the suggested new format is not as natural, however it won't require all
clients to recode.
If we really do want to migrate to an array, then a new selector might be the
way.
eg ..../node.aclarray.json
WDYT ?
Ian
1 http://svn.apache.org/viewvc?view=revision&revision=927532
"
After leaving this for a few days, there appears to be no objection to changing the format back to the original with an additional sequence property.