- From: Gregor Karlinger <gregor.karlinger@iaik.at>
- Date: Mon, 29 May 2000 11:10:56 +0200
- To: "Joseph M. Reagle Jr." <reagle@w3.org>, "XML" <w3c-ietf-xmldsig@w3.org>
Hi Joseph,
as I read your reply to Don's error report today, I recognized that I did
not
send you the list of errors I have found in the draft (for whatever reason
:-(
So here are the errors I have found; I hope you find it useful despite the
delay.
Section 1.3, last paragraph (macro definition):
I think the file extension in "xmldsig-core-schema.dtd" is misleading
here. Please replace it with ".xsd", as done in section 9.
Section 2.0, example:
Left parenthesis is missing before the Reference clause.
Section 2.1, example:
Alg ids for CanonicalicationMethod (line s03) and Transform (line s07)
are outdated.
Section 2.1, last paragraph:
Keyword OPTIONAL is misused here. Should be written in small letters.
Section 2.2, first paragraph:
Replace "use as foo:assuredby attribute" with "use a foo:assuredby
attribute"
Section 3.2.2:
Adapt section since there is a default c14n now.
Section 4.1, Schema definition:
Attribute Id should be declared with optional use.
Section 4.2, last sentence:
The Schema defintion for SignatureValue does not allow own encodings for
for
specified algorithms.
Section 4.3.1, first paragraph:
* Paragraph sould be adapted for default c14n.
* Refers to section "6.1 Algorithm Identifiers", should be "6.1 Algorithm
Identifiers and Implementation Requirements"
Section 4.3.3.1, third paragraph:
The words MimeType, Charset should be set in normal font, since they are
no longer
defined as attributes.
Section 4.3.3.1, Schema definition:
Why use mixed content Ttransform?
Section 4.3.3.1, DTD:
Reject MimeType and Charset attributes
Section 4.3.3.2, last sentence:
Reference to section 6.1 should be adapted (see my remark on section
4.3.1)
Section 4.4, Schema defintion:
* KeyInfo: maxOccurs in clause choice should be unbounded instead of 1
* KeyInfo <any> Element is missing in clause choice, otherwise it is not
possible
that applications place their own semantics within this element, as the
textual description says.
* KeyValue: Why mixed content model?
Section 4.4, DTD:
* Replace Element content model for KeyInfo by an internal entity to allow
for
application specific sematics.
* Content model for KeyName, RetrievalMethod, MgmtData is wrong
(%Key.ANY;)
Section 5.1, Schema definition:
* Why use <sequence> in type definition?
* Either minOccurs='1' is wrong or the content model of the corresponding
DTD Element
declaration (Reference*)
Section 6.1, first paragraph, third sentence:
Adapt sentence for default c14n.
Section 6.3, first paragraph:
Adapt paragraph for default c14n.
Section 6.4, first paragraph:
Adapt paragraph for default c14n.
Section 6.6, first paragraph:
Adapt paragraph, since MimeType and Charset attributes no longer exist.
Section 6.6.2:
Remove quoted printable.
Section 7.1
I think your remark in the first ordered list, item 2 actually belongs to
the second
ordered list, item 2.
Regards, Gregor
---------------------------------------------------------------
Gregor Karlinger
mailto://gregor.karlinger@iaik.at
http://www.iaik.at
Phone +43 316 873 5541
Institute for Applied Information Processing and Communications
Austria
---------------------------------------------------------------
Received on Monday, 29 May 2000 05:11:37 UTC