Re: [Ecrit] WGLC for draft-ietf-ecrit-similar-location - end date April 8

[Peter Saint-Andre <[e-mail]>]

On 3/25/18 2:49 AM, Az Mankin wrote:
> This starts WGLC for draft-ietf-ecrit-similar-location-05,**A LoST
> extension to return complete and similar location info
> 
> Please send both comments and messages that you support advancement to
> this mailing list before April 8.

Sorry about the slow response. Here is some feedback on the schema (each
comment is prefaced by "^^^").

###

<?xml version="1.0" encoding="UTF-8"?>
<xs:schema xmlns:xs="http://www.w3.org/2001/XMLSchema"
           xmlns="urn:ietf:params:xml:ns:lost-rli1"
           targetNamespace="urn:ietf:params:xml:ns:lost-rli1"
           elementFormDefault="qualified">
     <!-- Import base Lost -->
     <xs:import namespace="urn:ietf:params:xml:ns:lost1"/>

^^^ We import this for the <location/> element, right? That should be
prefaced appropriate below...

<!-- extend the extensionPoint of commonRequestPattern of findService to
include:  -->

^^^ Although we can say where an extension is intended to go, AFAIK
there's no way in XML schema to restrict the use of an extension to a
particular place in a structure defined by another schema.

      <xs:atttribute name="asOf" type="xs:dateTime" minOccurs="0"/>

^^^ As Dan mentioned, this might have been copied from another schema.

      <xs:attribute name="returnAdditionalLocation" minOccurs="0">
       <xs:simpleType>
          <xs:restriction base="xs:token">
            <xs:enumeration value="none"/>
            <xs:enumeration value="similar"/>
            <xs:enumeration value="complete"/>
            <xs:enumeration value="any"/>
          </xs:restriction>
        </xs:simpleType>
      </xs:attribute>

<!-- extend the extensionPoint of commonResponsePattern of
findServiceReponse to include -->
 <xs:group ref="similarLocation"/>
        <xs:group ref="completeLocation"/>
<!-- where -->
<xs:group name="similarLocation">
    <xs:sequence xmlns:base="urn:ietf:params:xml:ns:lost1">

^^^ add the imported namespace here

      <xs:element ref="base:location" maxOccurs="unbounded"/>

^^^ add the namespace preface

      <xs:any namespace="##other" processContents="lax"
          minOccurs="0" maxOccurs="unbounded"/>
    </xs:sequence>
  </xs:group>

<xs:group name="completeLocation">
    <xs:sequence xmlns:base="urn:ietf:params:xml:ns:lost1">
      <xs:element ref="base:location" maxOccurs="1"/>

^^^ as above

      <xs:any namespace="##other" processContents="lax"
          minOccurs="0" maxOccurs="unbounded"/>
    </xs:sequence>
  </xs:group>
</xs:schema>

###

Because the definitions of the similarLocation and completeLocation
groups are the same, we could define this in a less verbose way, but
what we have is OK.

Peter