[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: AD review: draft-ietf-netconf-soap-05
Comments are inline:
(Bert, thanks for the review and the XML Registry clarification;
I've incorporated those suggestions below as well.)
On 1-Sep-05, at 1:32 PM, Wijnen, Bert (Bert) wrote:
Hre is my AD review. Sorry that this one slipped through
the cracks and took a bit longer. I have also seen
the suggested changes (as per below email from Ted) and
will assume such changes will be made.
A lot of the SOAP/WSDL stuff is completely new for me,
so pls bear with me if I ask newbee questions.
But I think we should assume that others who read the
documents may also be newbees, and so maybe we should at
least make it all clear in the document.
- I do not see in the document how a session-id gets
passed from setting up a BEEP or HTTP conenction
to the NetConf protocol level so that NetConf can use
the session-id. Maybe it is there, but I do not see it.
Pls explain and clarify in the text.
This is exchanged through <hello> messages. The opening sentence
in 3.3 will be revised as follows:
Capabilities exchange and session ID establishment are performed
through the exchange of <hello> messages.
- sect 3.7
- Is it OK to assume a "hypothetical location", or should we
decide and/or request IANA or xmlsoap.org to define a
real and approved location?
IANA should be requested to provide a location for the NETCONF
schema (in netconf-prot, I believe). How should netconf-soap
be revised to accommodate an IANA request in netconf-prot?
- Have you (has anyone) done a SYNTAX check on the WSDL
document, and if so, which tool was used?
The WSDL has been checked with xmllint against the xmlsoap.org
schemas.
- In the WSDL document, there are www.iana.org items.
for example I see:
<import namespace="urn:ietf:params:xml:ns:netconf:base:1.0"
location="http://www.iana.org/assignments/xml-registry/
schema/netconf-base_1.0.xsd"/>
WHere is that coming from.
Maybe that is to be found in the missing (i.e. TBD) text from
the netconf-protocol document?
Both the namespace and the netconf schema location should be
requested from IANA in the netconf-protocol document. The
netconf-soap draft will be revised to request a netconf-soap
namespace.
The IANA is requested to assign an XML namespace for the NETCONF
over SOAP WSDL definitions.
Another example
xmlns:SOAP="http://schemas.xmlsoap.org/wsdl/soap/";
Does that need a citation/reference?
This is the standard SOAP namespace as defined in the WSDL reference.
xmlns:tns="urn:ietf:params:xml:ns:netconf:soap:1.0"
Does IANA now how to create that? Or is there no IANA
action needed for that?
IANA action is needed (request described above).
There may be other stuff. Once I understand the above I can
check further and see if I have more questions.
- IANA considerations Section
- says "IANA will". Maybe better: "IANA is requested to"
- Is the port for BEEP not already requested via the
netconf-over-beep document?
This is a different usage of BEEP. netconf-beep does not use
the SOAP BEEP mapping (rfc3288bis) but rather uses BEEP directly.
- Where is IANA supposed to put the WSDL definition?
you say in the XML registry. WHich is the document that
explains how to do that? Where is the ptr to the iana
we page that contains these sort of documents?
This IANA request will be removed.
CItation/Reference issues:
!! Missing citation for Normative reference:
P018 L030: [5] Freed, N. and N. Borenstein, "Multipurpose
Internet Mail
!! Missing citation for Normative reference:
P018 L034: [6] Freed, N. and N. Borenstein, "Multipurpose
Internet Mail
!! Missing citation for Normative reference:
P019 L014: [13] Rose, M. and D. New, "Reliable Delivery for
syslog", RFC 3195,
!! Missing citation for Informative reference:
P019 L035: [18] Barton, J., Nielsen, H. and S. Thatte, "SOAP
Messages with
!! Missing citation for Informative reference:
P019 L045: [20] Nadalin, A., Kaler, C., Hallam-Baker, P. and
R. Monzillo, "Web
Extraneous references removed.
NITS:
- Pls expand Acrionyms first time they are used.
For example BEEP in the abstract.
WSDL in 3rd para section 1.
Expanded.
- Sect 2.5, last sentence.
you may want to add a citation (and reference) to the RFC(s)
that describes "chunking and persistent connections".
Citation added (HTTP 1.1).
- I see sometimes "Reciever" and other times "Receiver".
I think it should be consistent: "Receiver"
- Security COnsiderations section: s/IPSec/IPsec/
Fixed.
Thanks,
Ted.
Bert
-----Original Message-----
From: owner-netconf@ops.ietf.org [mailto:owner-
netconf@ops.ietf.org]On
Behalf Of Ted Goddard
Sent: Tuesday, August 30, 2005 19:46
To: netconf
Subject: proposed changes for draft-ietf-netconf-soap-06
Hi All,
I would like to propose the following changes for the NETCONF SOAP
draft in the indicated sections:
Section 0
RFC 3978 boilerplate
Section 2.4 BCP56: On the Use of HTTP as a Substrate
It is also possible to respond to the concern on the re-use of
port 80. A NETCONF SOAP service SHOULD be offered over a new
standard port for NETCONF over SOAP (over HTTP) to
be defined as requested in the IANA considerations of this
document.
Section 4 Security Considerations
The IANA requested port SHOULD be used, as this provides a
means
for efficient firewall filtering during possible
denial-of-service
attacks.
Section 5 IANA Considerations
The IANA is requested to assign TCP ports for NETCONF for SOAP
over HTTP and SOAP over BEEP.
The IANA is requested to place netconf-soap_1.0.wsdl in the
IANA XML registry.
The following indicated ID-nits appear to be in error (xml2rfc
output checked with "od -c"):
tmp/draft-ietf-netconf-soap-05.txt(452):
Line is too long: the offending characters are 'elope"'
tmp/draft-ietf-netconf-soap-05.txt(464):
Line is too long: the offending characters are
's:netconf:base:1.0">'
Thanks,
Ted.
--
to unsubscribe send a message to netconf-request@ops.ietf.org with
the word 'unsubscribe' in a single line as the message text body.
archive: <http://ops.ietf.org/lists/netconf/>
--
to unsubscribe send a message to netconf-request@ops.ietf.org with
the word 'unsubscribe' in a single line as the message text body.
archive: <http://ops.ietf.org/lists/netconf/>