James,

Many thanks for the review. Your clarifications are reasonable to me.

With regard to JSON names vs JSON members, the intent was to draw the
distinction to the name vs the entire name and value. Regardless, I'll
look into clarifying the concepts and some better explanatory text
both and synchronization with the current RFCS.

Thanks.
-andy

On Mon, Jun 26, 2023 at 9:32 AM Gould, James <jgo...@verisign.com> wrote:
>
> Andy,
>
>
>
> Thanks for creating the draft, below is my feedback:
>
>
>
> Section 2 “RDAP Extension Identifier”
>
> It would be better to refer to JSON members instead of JSON attribute names 
> to be more consistent with RFC 9083.
> I would make it clear that the extension identifier can prepend or match the 
> path segments and JSON members, such as the use of “redacted” extension 
> identifier in draft-ietf-regext-rdap-redacted that is used for the “redacted” 
> JSON member of the extension.  Prepending does support matching, but I would 
> be more explicit.
>
> Section 3 “Usage in Queries”
>
> Same feedback from section 2, where the extension identifiers prepend or 
> match the path segments.  In the case of multiple path segments, you can 
> separate the extension identifier from the path segment value with an 
> underbar (e.g., foobar_fizz and foobar_fazz) in the example you provide.
>
> Section 4 “Usage in JSON”
>
> It would be better to refer to JSON members instead of JSON names.
> Same feedback from section 2, where the extension identifiers prepend or 
> match the JSON members.  The RDAP versioning extension 
> (draft-gould-regext-rdap-versioning) provides an example of matching the 
> extension identifier in the query response with “versioning” and use of the 
> extension identifier as a prefix for the help response (should be 
> “versioning_help” instead of “versioning-help”).
>
> Section 5 “Camel Casing”
>
> It would be better to refer to JSON members instead of JSON names.
>
> Section 7 “Extension Versioning“
>
> I would change the section title to “Versioning in Extension Identifiers”, 
> since extensions can certainly have versioning, but the extension identifiers 
> do not include any explicit versioning.
> I would change the references in the section of “RDAP extensions” to “RDAP 
> extension identifiers”.  For example, “That is, the RDAP extension 
> identifiers are opaquely versioned”.
>
>
>
> Thanks,
>
>
>
> --
>
>
>
> JG
>
>
>
>
>
>
>
> James Gould
>
> Fellow Engineer
>
> jgo...@verisign.com 
> <applewebdata://13890C55-AAE8-4BF3-A6CE-B4BA42740803/jgo...@verisign.com>
>
>
>
> 703-948-3271
>
> 12061 Bluemont Way
>
> Reston, VA 20190
>
>
>
> Verisign.com <http://verisigninc.com/>
>
>
>
>
>
>
>
>
>
> On 5/21/23, 10:36 PM, "regext on behalf of Andrew Newton" 
> <regext-boun...@ietf.org <mailto:regext-boun...@ietf.org> on behalf of 
> a...@hxr.us <mailto:a...@hxr.us>> wrote:
>
>
>
>
>
> Caution: This email originated from outside the organization. Do not click 
> links or open attachments unless you recognize the sender and know the 
> content is safe.
>
>
>
>
>
> Hi all,
>
>
>
>
>
> Jasdip and I have put together a draft for a new RDAP media type. This
>
> allows clients to signal their supported extensions. By using a media
>
> type, this signaling survives redirects and is backward compatible
>
> with the RDAP RFCs and the currently deployed RDAP ecosystem.
>
>
>
>
>
> We even have some demo code to show how it works:
>
> https://secure-web.cisco.com/1vRQc9yIEtZ13eYnKDqW81F4k6q7cchX3N8mZFqkP5Yabk4J8yOsdQ6XUzS9wYEsHarDVy5m_i0VIwJnfzEfCeU3qf3sL4C-MCl2Kw_50BwWS2VUJ4pahCMhbiHRa5Chfl-lzAguOVvLdJZuhfuSr8OF06Qm7vLWaUl89-BGh3Jh6RNsBF4aGqC8IICTceEeiiqW3X5fnifXnJh5e0ZapDM7SRwjhUt_D0hs8MfTcHKQtdbDrh4CMVqMHrTEVJhz8KI0Mf-UwuEM7NABHQbjUM_URrFC61gpYw1MR7QXfLtQ/https%3A%2F%2Fgithub.com%2Fanewton1998%2Fdraft-regext-ext-json-media-type%2Ftree%2Fmain%2Fdemo
>  
> <https://secure-web.cisco.com/1vRQc9yIEtZ13eYnKDqW81F4k6q7cchX3N8mZFqkP5Yabk4J8yOsdQ6XUzS9wYEsHarDVy5m_i0VIwJnfzEfCeU3qf3sL4C-MCl2Kw_50BwWS2VUJ4pahCMhbiHRa5Chfl-lzAguOVvLdJZuhfuSr8OF06Qm7vLWaUl89-BGh3Jh6RNsBF4aGqC8IICTceEeiiqW3X5fnifXnJh5e0ZapDM7SRwjhUt_D0hs8MfTcHKQtdbDrh4CMVqMHrTEVJhz8KI0Mf-UwuEM7NABHQbjUM_URrFC61gpYw1MR7QXfLtQ/https%3A%2F%2Fgithub.com%2Fanewton1998%2Fdraft-regext-ext-json-media-type%2Ftree%2Fmain%2Fdemo>
>
>
>
>
>
> -andy
>
>
>
>
>
> ---------- Forwarded message ---------
>
> From: <internet-dra...@ietf.org <mailto:internet-dra...@ietf.org>>
>
> Date: Sun, May 21, 2023 at 10:26 PM
>
> Subject: New Version Notification for
>
> draft-newton-regext-rdap-x-media-type-00.txt
>
> To: Andy Newton <a...@hxr.us <mailto:a...@hxr.us>>, Jasdip Singh 
> <jasd...@arin.net <mailto:jasd...@arin.net>>
>
>
>
>
>
>
>
>
>
>
>
>
>
> A new version of I-D, draft-newton-regext-rdap-x-media-type-00.txt
>
> has been successfully submitted by Andy Newton and posted to the
>
> IETF repository.
>
>
>
>
>
> Name: draft-newton-regext-rdap-x-media-type
>
> Revision: 00
>
> Title: An RDAP With Extensions Media Type
>
> Document date: 2023-05-21
>
> Group: Individual Submission
>
> Pages: 10
>
> URL:
>
> https://secure-web.cisco.com/1Zy0H8UzvzEBBDkFv-crlCZ8_sbHpdKDUvCU-z9pyKitatAG9921vRsmo8kETZ8E9mfa0vu707fwKtgpPmLhOo1n5Zb0DChdROrujLnm9ZkU0bB8PUB5bsyXQjkfyQ0F2QNRJI8yzoo6U7P6_xTc9wCJBgHHCGXU6BW2QnxBLBy0G5N1oBJkqdsqWCry_o43GjD-Z8jrA0-5imMF6rrzUYKVOH5Tj4qlbpJrn-299zrkzbHonQKLtBwRtlL2Af2rZPJzHxJXlrazvBBtx_oH_mcMble7PO6wWSZYo7l5930o/https%3A%2F%2Fwww.ietf.org%2Farchive%2Fid%2Fdraft-newton-regext-rdap-x-media-type-00.txt
>  
> <https://secure-web.cisco.com/1Zy0H8UzvzEBBDkFv-crlCZ8_sbHpdKDUvCU-z9pyKitatAG9921vRsmo8kETZ8E9mfa0vu707fwKtgpPmLhOo1n5Zb0DChdROrujLnm9ZkU0bB8PUB5bsyXQjkfyQ0F2QNRJI8yzoo6U7P6_xTc9wCJBgHHCGXU6BW2QnxBLBy0G5N1oBJkqdsqWCry_o43GjD-Z8jrA0-5imMF6rrzUYKVOH5Tj4qlbpJrn-299zrkzbHonQKLtBwRtlL2Af2rZPJzHxJXlrazvBBtx_oH_mcMble7PO6wWSZYo7l5930o/https%3A%2F%2Fwww.ietf.org%2Farchive%2Fid%2Fdraft-newton-regext-rdap-x-media-type-00.txt>
>
> Status:
>
> https://secure-web.cisco.com/1WoZczv98b_VYLPdW3niQTAXGicG11lNmRaYIArEIkaq0g7S2Y-ya2dKkpZ4x5486FeKIA2gS9YSh7FNfbWeDKxZMr0tjfyXQdTpP2OYN1nsOzxzMbLGRzk18XH7v2s91gwr6EpQrFNuSwykYmUUDAnqQCWTAWc6pQXtSz68Ysccu1L871hc04bDsHJKQ9TYTM8gLLoVpMMGtbtipNr7rMWJmLJ0EAR9Q9MvUuOWT7vqy_koggS0DZVOiNaXC9TP8fhv7rfjX87ouJtS4uJXZTU3MolxT8l-uo-wwDRfvhT0/https%3A%2F%2Fdatatracker.ietf.org%2Fdoc%2Fdraft-newton-regext-rdap-x-media-type%2F
>  
> <https://secure-web.cisco.com/1WoZczv98b_VYLPdW3niQTAXGicG11lNmRaYIArEIkaq0g7S2Y-ya2dKkpZ4x5486FeKIA2gS9YSh7FNfbWeDKxZMr0tjfyXQdTpP2OYN1nsOzxzMbLGRzk18XH7v2s91gwr6EpQrFNuSwykYmUUDAnqQCWTAWc6pQXtSz68Ysccu1L871hc04bDsHJKQ9TYTM8gLLoVpMMGtbtipNr7rMWJmLJ0EAR9Q9MvUuOWT7vqy_koggS0DZVOiNaXC9TP8fhv7rfjX87ouJtS4uJXZTU3MolxT8l-uo-wwDRfvhT0/https%3A%2F%2Fdatatracker.ietf.org%2Fdoc%2Fdraft-newton-regext-rdap-x-media-type%2F>
>
> Html:
>
> https://secure-web.cisco.com/1CK0G-mVgqHY1XgneRSalGr_hQp0MOEb27ml_Ba7dT7oKiufTSkh-BdeMkDhWXOIa7ItaVgkthVFnI6XvtBQZJGK04aN4ixpBSbE2_wWNddHxsG3m3xLlrDr6C-cLEbPx9TwnpY1x3cJSsbKQ5LJEFg6YFcTX8E2IfRq-gPVTfhL5wXKNxi4E_n79_Nc2X7r8fuNucGSon8gLvWSezqFfIxUDtgXf69-Fc6v5hsIVMoxRb3obS9mBelOw-9h-hnkYafS08SckCEiUe7UUR8JknKNlHT_jZ5tUPA81BmWcvro/https%3A%2F%2Fwww.ietf.org%2Farchive%2Fid%2Fdraft-newton-regext-rdap-x-media-type-00.html
>  
> <https://secure-web.cisco.com/1CK0G-mVgqHY1XgneRSalGr_hQp0MOEb27ml_Ba7dT7oKiufTSkh-BdeMkDhWXOIa7ItaVgkthVFnI6XvtBQZJGK04aN4ixpBSbE2_wWNddHxsG3m3xLlrDr6C-cLEbPx9TwnpY1x3cJSsbKQ5LJEFg6YFcTX8E2IfRq-gPVTfhL5wXKNxi4E_n79_Nc2X7r8fuNucGSon8gLvWSezqFfIxUDtgXf69-Fc6v5hsIVMoxRb3obS9mBelOw-9h-hnkYafS08SckCEiUe7UUR8JknKNlHT_jZ5tUPA81BmWcvro/https%3A%2F%2Fwww.ietf.org%2Farchive%2Fid%2Fdraft-newton-regext-rdap-x-media-type-00.html>
>
> Htmlized:
>
> https://secure-web.cisco.com/1GrhyTTE-K0oFVSgLSDCxO_XZXFH4RvCkHZsBNJN6-IH46FewVDHfZtsrvAj-EFR_2Czu_Oq1yxqg4AorisBnyu7MHgX9T8IQivu2jjSg3juhs0uMzhkaPCdPJbsz9MG71KMoLGjXD1cdeXhXwrplTZwxsWciqhu0AvtewbXQmfOCJHtHfZIrulOYs-e2PdbEO_LVoiIBYdGuMC3tUia-fMoFeddmfgemevKCWmrRqDVYSBGT6R6DFvh28WYMzXDK-MI4zJvoJBu7uFf40pIgqv3fRLhR5Kp38z5Cf13yz04/https%3A%2F%2Fdatatracker.ietf.org%2Fdoc%2Fhtml%2Fdraft-newton-regext-rdap-x-media-type
>  
> <https://secure-web.cisco.com/1GrhyTTE-K0oFVSgLSDCxO_XZXFH4RvCkHZsBNJN6-IH46FewVDHfZtsrvAj-EFR_2Czu_Oq1yxqg4AorisBnyu7MHgX9T8IQivu2jjSg3juhs0uMzhkaPCdPJbsz9MG71KMoLGjXD1cdeXhXwrplTZwxsWciqhu0AvtewbXQmfOCJHtHfZIrulOYs-e2PdbEO_LVoiIBYdGuMC3tUia-fMoFeddmfgemevKCWmrRqDVYSBGT6R6DFvh28WYMzXDK-MI4zJvoJBu7uFf40pIgqv3fRLhR5Kp38z5Cf13yz04/https%3A%2F%2Fdatatracker.ietf.org%2Fdoc%2Fhtml%2Fdraft-newton-regext-rdap-x-media-type>
>
>
>
>
>
>
>
>
>
> Abstract:
>
> This document defines a media type for RDAP that can be used to
>
> describe RDAP content with RDAP extensions. Additionally, this
>
> document describes the usage of this media type with RDAP.
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
> The IETF Secretariat
>
>
>
>
>
> _______________________________________________
>
> regext mailing list
>
> regext@ietf.org <mailto:regext@ietf.org>
>
> https://secure-web.cisco.com/1C8sLMltcoF0Tdh1Plg0qpBO41PST7VqVTdQ78Fpf0CWGetzu6l6kIDM8z78TAxAbZjU6YQfHuTcxY9gtuK4REIMKdRCwXQJlsF1qXtmHD32e5_0saOFKCkhPfGctMbCzZTBcO3JHHN1vhWiTmUZPLpGEuvMK_SGnZEJSjaoACxo-eWXochUg1pgfSvs5FS9d-4CQk_3gOBn6xXuP0fBA0kloYpoPMqJMM40AZpZBVA-x4pZEbTtyxfKo6ajiquaBC4RhxJSoPGBgvSMQ-tlHFW_jjUzjiW4ANuJYrg11Ens/https%3A%2F%2Fwww.ietf.org%2Fmailman%2Flistinfo%2Fregext
>  
> <https://secure-web.cisco.com/1C8sLMltcoF0Tdh1Plg0qpBO41PST7VqVTdQ78Fpf0CWGetzu6l6kIDM8z78TAxAbZjU6YQfHuTcxY9gtuK4REIMKdRCwXQJlsF1qXtmHD32e5_0saOFKCkhPfGctMbCzZTBcO3JHHN1vhWiTmUZPLpGEuvMK_SGnZEJSjaoACxo-eWXochUg1pgfSvs5FS9d-4CQk_3gOBn6xXuP0fBA0kloYpoPMqJMM40AZpZBVA-x4pZEbTtyxfKo6ajiquaBC4RhxJSoPGBgvSMQ-tlHFW_jjUzjiW4ANuJYrg11Ens/https%3A%2F%2Fwww.ietf.org%2Fmailman%2Flistinfo%2Fregext>
>
>
>
>

_______________________________________________
regext mailing list
regext@ietf.org
https://www.ietf.org/mailman/listinfo/regext

Reply via email to