From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 9440DA00C4; Tue, 26 Jul 2022 16:36:23 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 3D94840E0F; Tue, 26 Jul 2022 16:36:23 +0200 (CEST) Received: from smartserver.smartsharesystems.com (smartserver.smartsharesystems.com [77.243.40.215]) by mails.dpdk.org (Postfix) with ESMTP id 6B5EA40697 for ; Tue, 26 Jul 2022 16:36:22 +0200 (CEST) X-MimeOLE: Produced By Microsoft Exchange V6.5 Content-class: urn:content-classes:message MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Subject: RE: [PATCH v2 00/13] telemetry JSON escaping and other enhancements Date: Tue, 26 Jul 2022 16:36:21 +0200 Message-ID: <98CBD80474FA8B44BF855DF32C47DC35D871EC@smartserver.smartshare.dk> In-Reply-To: <20220725163543.875775-1-bruce.richardson@intel.com> X-MS-Has-Attach: X-MS-TNEF-Correlator: Thread-Topic: [PATCH v2 00/13] telemetry JSON escaping and other enhancements Thread-Index: AdigRKAeMmOUd54LRhepTWc7uuj9DwAuDzjA References: <20220623164245.561371-1-bruce.richardson@intel.com> <20220725163543.875775-1-bruce.richardson@intel.com> From: =?iso-8859-1?Q?Morten_Br=F8rup?= To: "Bruce Richardson" , X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org > From: Bruce Richardson [mailto:bruce.richardson@intel.com] > Sent: Monday, 25 July 2022 18.35 >=20 > This patchset contains fixes for the problem of handling characters > returned by telemetry callbacks which require escaping when encoded in > JSON format. It also includes unit tests to validate the correct > encoding in such scenarios and a number of smaller enhancements to > telemetry and telemetry testing. >=20 > RFC->V2: > * limited characters allowed in dictionary element names and command > names to side-step the encoding problems there. > * added support for proper escaping of dictionary string values > * added more testing and test cases > * added other misc telemetry cleanups and refactoring >=20 Good job, Bruce! Series-Acked-by: Morten Br=F8rup