[ https://jira.jboss.org/jira/browse/DNA-513?page=com.atlassian.jira.plugin... ] Randall Hauch updated DNA-513: ------------------------------ Priority: Blocker (was: Major) Changed to BLOCKER, since this is a pretty big deal. Brian has started looking at it, but I'll also take a look. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/jira/browse/DNA-513?page=com.atlassian.jira.plugin... ] Randall Hauch commented on DNA-513: ----------------------------------- I talked with Brian about this, and perhaps the most challenging aspect of this is knowing which properties (if any) are encoded in a client PUT/POST request. Either every property value has to be looked at as possibly being encoded, or there has to be some annotation in the request defining whether a property value is encoded. The former is expensive and potentially error prone (e.g., if a string property value is stored already encoded in the repository, or if a string property coincidentally matches an encoded representation). The latter approach is less attractive, but is really the only surefire way for the client and server to know which properties have been encoded in the request and must be decoded prior to use. There seem to be two options for annotating the properties: include the annotation in the name or in the value. Annotating the name is quite straightforward as long as the annotation includes characters that are not normally allowed in a property name (so we always know that the annotation is what it is, and not coincidentally part of the property name). Annotating the value appears on the surface to be pretty easy, but multi-valued properties and the wider range of values allowed make it more challenging and potentially more risky. Therefore, the approach we're going to take is to annotation the property name to specify that the property value(s) has been encoded. The format will be "{propertyName}/{encoding}/", where "{propertyName}" is the usual name of the property, and "{encoding}" is the lowercase name of the encoding that is used. For example, a "jcr:data" property encoded in Base64 would be represented in the JSON representations as "jcr:data/base64/". According to the JCR 2.0 PFD Section 3.2.2, the local name (which applies to property names) can only contain valid XmlChar except '/' | ':' | '[' | ']' | '|' | '*' (where the JCR 1.0 spec is a bit more lenient, and it basically says that a name is not valid if it begins or ends with a '/', ':', '[', ']', '*', ''', '"', '|' or any whitespace character). Therefore the {propertyName}/{encoding}/" format is never a valid property name, so can always be used to determine the encoding. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/jira/browse/DNA-513?page=com.atlassian.jira.plugin... ] Work on DNA-513 started by Randall Hauch. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/jira/browse/DNA-513?page=com.atlassian.jira.plugin... ] Brian Carothers commented on DNA-513: ------------------------------------- This patch looks great. Thanks for handling this, Randall! -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira