Re: [apps-discuss] Feedback on draft-ietf-appsawg-json-patch-00
"Murray S. Kucherawy" <msk@cloudmark.com> Fri, 02 March 2012 22:50 UTC
Return-Path: <msk@cloudmark.com>
X-Original-To: apps-discuss@ietfa.amsl.com
Delivered-To: apps-discuss@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 1F10221F8568 for <apps-discuss@ietfa.amsl.com>; Fri, 2 Mar 2012 14:50:46 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.604
X-Spam-Level:
X-Spam-Status: No, score=-102.604 tagged_above=-999 required=5 tests=[AWL=-0.005, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id FpwglCtcrnmD for <apps-discuss@ietfa.amsl.com>; Fri, 2 Mar 2012 14:50:45 -0800 (PST)
Received: from ht1-outbound.cloudmark.com (ht1-outbound.cloudmark.com [72.5.239.25]) by ietfa.amsl.com (Postfix) with ESMTP id DD32D21E806D for <apps-discuss@ietf.org>; Fri, 2 Mar 2012 14:50:44 -0800 (PST)
Received: from EXCH-MBX901.corp.cloudmark.com ([fe80::addf:849a:f71c:4a82]) by EXCH-HTCAS901.corp.cloudmark.com ([fe80::2966:6846:8d89:4681%12]) with mapi id 14.01.0355.002; Fri, 2 Mar 2012 14:50:44 -0800
From: "Murray S. Kucherawy" <msk@cloudmark.com>
To: "apps-discuss@ietf.org" <apps-discuss@ietf.org>
Thread-Topic: [apps-discuss] Feedback on draft-ietf-appsawg-json-patch-00
Thread-Index: AQHM9+d/ykpyZt+42kKhe+gEF12z9JZWfmmAgAGb5wD//4MRcA==
Date: Fri, 02 Mar 2012 22:50:43 +0000
Message-ID: <9452079D1A51524AA5749AD23E003928077040@exch-mbx901.corp.cloudmark.com>
References: <4F4FD7EB.8080004@cloudmark.com> <1330638087.2531.7.camel@neutron> <4F51468E.4000006@cloudmark.com>
In-Reply-To: <4F51468E.4000006@cloudmark.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [172.20.2.121]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Subject: Re: [apps-discuss] Feedback on draft-ietf-appsawg-json-patch-00
X-BeenThere: apps-discuss@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: General discussion of application-layer protocols <apps-discuss.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/apps-discuss>
List-Post: <mailto:apps-discuss@ietf.org>
List-Help: <mailto:apps-discuss-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 02 Mar 2012 22:50:46 -0000
> -----Original Message----- > From: apps-discuss-bounces@ietf.org [mailto:apps-discuss-bounces@ietf.org] On Behalf Of Mike Acar > Sent: Friday, March 02, 2012 2:16 PM > To: apps-discuss@ietf.org > Subject: Re: [apps-discuss] Feedback on draft-ietf-appsawg-json-patch-00 > > >> So to avoid depending upon undefined behavior, an implementation of > >> Patch would have to parse the Pointer, remove the "target" value from > >> the end, and then traverse the JSON objects to find the parent of the > >> target. > > > > These specifications do very little to define how to handle errors > > beyond terminating processing and indicating an error condition. > > I don't think Pointer can even say this MAY be an error: that's one of > the semantics an application assigns to Pointers (cf > http://www.ietf.org/mail-archive/web/apps- > discuss/current/msg04411.html). > > For Patch, in my opinion it MUST be an error if a Pointer reference > token can't be found in an object or array - but the consequence of > that is that pointers have to point to parents. +1. This points (hah) back to the pointer document thread: That document can leave handling of failed pointer resolution explicitly undefined, but the application has to say what semantics are to be applied in such cases. For the patch document, it MUST be an error. -MSK
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Paul C. Bryan
- [apps-discuss] Feedback on draft-ietf-appsawg-jso… Mike Acar
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Murray S. Kucherawy
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Mike Acar
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Julian Reschke
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Murray S. Kucherawy
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Paul C. Bryan
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Murray S. Kucherawy
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Julian Reschke
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Julian Reschke
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Murray S. Kucherawy
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Julian Reschke
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Murray S. Kucherawy
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Mike Acar
- Re: [apps-discuss] Feedback on draft-ietf-appsawg… Murray S. Kucherawy