Re: [apps-discuss] Review of draft-ietf-appsawg-json-pointer

On Thu, Nov 15, 2012 at 7:24 AM, Paul Hoffman <paul.hoffman@vpnc.org> wrote:

> This document mostly looks fine.
>
> - I do not understand the use of "-"; at least one use of it in an
> application should be given.
>
> - If "-" is useful, wouldn't it also be useful to have something that
> indicates the (non-existant) member before the first array element?
>
>
This came from a requirement with json patch to allow a way of appending
new array items to the end. This was fairly extensively discussed and
concluded that the - was the least-complicated option that met the
requirement.

- James

> - Editorial nit in section 4:
>       *  characters that represent an unsigned base-10 integer value
>          (possibly with leading zeros), making the new referenced value
>          is the array element with the zero-based index identified by
>          the token, or
> That should either be "the new referenced value is the" or "making the new
> referenced value the", I believe.
>
> - One of the security considerations is too limited.
>    Note that JSON pointers can contain the NUL (Unicode U+0000)
>    character, which may not be representable in all programming
>    languages.
> The same is true for all control characters, and for some programming
> languages, all non-ASCII characters. Proposed rewording:
>    Note that JSON pointers can contain the NUL (Unicode U+0000)
>    character, control characters, non-ASCII characters, and so
>    on. These characters may not be representable in all programming
>    languages.

> --Paul Hoffman
> _______________________________________________
> apps-discuss mailing list
> apps-discuss@ietf.org
> https://www.ietf.org/mailman/listinfo/apps-discuss
>