On Tue, 30 Nov 2004, Oldroyd, Bill wrote:

> I have made available a test SRU Service at the following URL :
> http://herbie.bl.uk:9080/cgi-bin/blils.cgi
> Comments welcome.

Some initial comments, now that I've had time to look at it.

Explain:

* Your indexes are all very generic DC indexes, but you have them in a new
context-set called 'tel'.  While this is supported, it's generally
considered to be bad form to redefine a whole slew of indexes without
a good reason.

Is there a reason why, for example, tel.creator isn't just dc.creator, and
tel.server isn't cql.serverChoice?

I do note (by trial and error) that you support dc.title

* Your schemas have invalid identifiers.  At the very least they need to
be URIs.  What you have in them currently should be in the name attribute.

General:

* You don't process the version request param at all.
* You don't process recordPacking ...
* ... nor recordXPath.  You don't fail the requests, but do return the
   appropriate diagnostic.
* You claim to be SRU version 1.0, but return extraResponseData which is
   1.1

None of these are too bad, however ...

* You return extraResponseData without it being requested.  This is
specifically forbidden in SRW/U.  There -must- be a parameter present in
the request in order to return this information.

* Your namespacing in the XML is also incorrect.
For example you have <SRW:numberOfRecords> but then <version> for the SRW
version field and <version> for your own extraData version field.
You also don't have a default namespace.

* Matthew may correct me, but I believe that extraResponseData must come
at the end, not the beginning, of the response.  Matthew?

Hope this helps,

Rob

       ,'/:.          Dr Robert Sanderson ([log in to unmask])
     ,'-/::::.        http://www.o-r-g.org/~azaroth/
   ,'--/::(@)::.      Dept. of Computer Science, Room 805
,'---/::::::::::.    University of Liverpool
____/:::::::::::::.  L5R Shop: http://www.cardsnotwords.com/
I L L U M I N A T I