Re: CVS update: subversion/subversion/libsvn_delta xml_output.c

Greg Hudson <ghudson@MIT.EDU> writes:

> Karl wrote, regarding property values:
> > Wait a minute -- we're not base64-encoding them now? What if
> > someone sends a property value that includes "<" in its text? :-)
>
> We do svn_xml_escape_string on them, so "<" would be properly encoded
> as "<". (However, high-bit values or 0-bytes are a problem, so we
> should consider base64-encoding instead.)

For now, because properties are still "toys", I think it's okay to
list the following restrictions on them:

   * they must not be too large; each property name/value pair must
     easily fit into memory

   * they must not be binary data.

Later on we'll implement them with vim and vigor, just like
text-deltas.
Received on Sat Oct 21 14:36:12 2006