PaceServiceError comments

[Bill de hÓra <bill@xxxxxxxxxx>]

Hi,

Some comments on PaceServiceError.


[[[
Resources served with an application/atom+xml media type SHOULD be 
served with an X-Atom-Error HTTP header, whose value is the ErrorURI 
for that resource.
]]]

I see the example uses a relative URI. Please specify that this URI 
be absolute.


[[[
The <link> element SHOULD be placed as close as possible to the 
beginning of the document.
]]]

I would strike this as superfluous.


[[[
In addition to an X-Atom-Error HTTP header, Atom documents MAY 
include a <link> element with a @rel of 'service.error'. Although 
servers SHOULD include an X-Atom-Error HTTP header, an ErrorURI 
given solely in a <link> element may be used in its absence. If both 
an X-Atom-Error HTTP header and a <link> element are present, the 
value of the header and the <link> element's @href MUST be identical.
]]]

I wonder if both a header and a link are needed. In any case 
"identical" above should be defined as "identical per {XML|HTTP spec 
here}". I'm not fussed about which one, just that one is specified.


[[[
The request SHALL be a GET. No other verbs are currently specified 
for this URI.
]]]

I think this is a job for POST. Given the semantics of the Pace, the 
client can't reasonably claim is does not intend a side effect. 
Idempotence is an orthogonal argument.


[[[
The request MUST include an X-Atom HTTP header with a value of error.
]]]

I would prefer a URI be used as the value and not an opcode (ie 
http://purl.org/atom/api/error or some such).


[[[
Information concerning the specific error encountered SHALL NOT 
accompany the request.
]]]

What is the rationale for this specification?


[[[
Once one ErrorURI request has been sent, another SHOULD NOT be sent 
until the feed in question has been requested and returned with HTTP 
status 200.
]]]

For this it should be specified what happens if the client fails to 
receive a response from the ErrorURI.


[[[
5.6.4 Response
]]]

The initial list of response are not the same as the ensuing 
sections. I don't see the need to specify 5xx errors beyond what 
HTTP already has to say.

[[[
Server to Client A (failing to close <feed> element)
]]]

Please specify that absolute URIs are to be used in the HTTP header.


Overall tho', good job.

cheers
Bill