Opened 8 years ago
Closed 7 years ago
#849 closed enhancement (wontfix)
Document ID in URL path for record pages
| Reported by: | Twan Goosen | Owned by: | Twan Goosen |
|---|---|---|---|
| Priority: | trivial | Milestone: | VLO-4.2 or later |
| Component: | VLO web app | Version: | |
| Keywords: | Cc: | go.sugimoto@oeaw.ac.at, teckart@informatik.uni-leipzig.de |
Description
Proposal by Go:
In terms of sustainability, it would be good to start discussing improving VLO record identifiers. I think we could improve a bit simply by removing system specific strings etc. For instance, https://vlo.clarin.eu/record/http_58__47__47_hdl.handle.net_47_10932_47_00-027B-8955-9EB1-8E01-F instead of https://vlo.clarin.eu/record?docId=http_58__47__47_hdl.handle.net_47_10932_47_00-027B-8955-9EB1-8E01-F
So make the document ID a path parameter instead of a query parameter. Redirect rules and/or redundant handlers within the application will have to be put in place in order to keep old links and bookmarks from breaking.
Change History (5)
comment:1 Changed 8 years ago by
| Cc: | teckart@informatik.uni-leipzig.de added |
|---|
comment:2 Changed 8 years ago by
| Milestone: | VLO-3.4 → VLO-4.0 or later |
|---|
comment:5 Changed 7 years ago by
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
After touching a related part of code, I realised that this was raised before but we decided to not use a path parameter for the document ID, as they often contain forward slashes, which obviously is problematic (these could be escaped or encoded but there doesn't appear to be a generic solution and it also doesn't necessarily make the URLs prettier).
Needs more thought/discussion, so taking it off of the 3.4 milestone.