#325: Glitch in web interface handler
------------------------+---------------------------------------------------
Reporter: skaplun | Owner: jcaffaro
Type: defect | Status: in_merge
Priority: major | Milestone:
Component: WebSearch | Version:
Resolution: | Keywords: opensearch
------------------------+---------------------------------------------------
Comment (by simko):
Note that `/search/` works well and serves the same content as `/search`.
Maybe we should do the same for others like `/yourbaskets/add/`. Though
this introduces non-canonicality, which is not good.
In the case of `/somefacility/someaction`, I agree that it makes sense to
return 404 for the trailing slash. In the case of `/somefacility` only,
it may be better to redirect, because people are generally used to that,
e.g. compare [[http://directory.google.com/Top/Computers/Software/]] and
[[http://directory.google.com/Top/Computers/Software]]. In other words,
for our URLs `/somefacility/someaction`, like `/yourbaskets/add` or
`/yourbaskets/display`, the first one looks like a directory and the
second one looks like a page, and I guess I'd propose to interpret
trailing slashes for all-but-last components of the URL, and return 404
for the last one.
Seems similar to Jerome's take on the matter.
--
Ticket URL: <http://invenio-software.org/ticket/325#comment:9>
Invenio <http://invenio-software.org>
