First of all, congrats to the WG for making good progress. While we
haven't had much progress, I'd be inclined to say that the WG is
leaning towards consensus on where this Pace has got to. Having said
that, I do have some comments on it.
I could live with this Pace but have some improvements to suggest.
-0.5 on SHOULD provide a body with 201. There are tons of plausible
applications where there's no noticeable benefit from doing this, and
I think that thus SHOULD is inappropriate. I would propose that the
sentence be reworded to say "When the server generates a response
with a status code of 201 ("Created"), it MAY also return a response
body, which, if provided, MUST be an Atom Entry Document representing
the newly-created resource."
8.3, the intent is fine but the language is misleading. I think that
it would be easier to understand if we had some defined terms, "media
resource" and (I just made this up) "media link entry".
8.3 Media resources and media link entries
An entry which links to a resource using the "src" attribute of the
atom:content element is referred to as a "Media link entry", and the
target of the "src" attribute as a "media resource". A media
resource is typically non-textual and a Media resource link is
typically used to provide metadata about it. Such an entry SHOULD
contain an atom:link element with a link relation of "edit-resource"
that contains the IRI used to modify the media resource. Deletion of
a media link entry SHOULD result in the deletion of the linked media
resource.
... or am I missing something?
8.3.1 Can we phrase this in the positive rather than the negative?
The server MAY use the content of the Title: header, as provided or
in a modified form, in constructing a title for the resource.
Then write 8.3.2 in terms of media link entries.
... The response includes a "Location" header indicating the URI of
the media link entry and a representation of that Entry in the body
of the response. The media link entry includes...
7.2.4 In app:accept, why do you have a magic "entry" value, why not
always use a media type and the default value is application/atom+xml?
-Tim
