[ https://jira.duraspace.org/browse/DS-640?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17856#action_17856 ]
Tim Donohue commented on DS-640: -------------------------------- Stuart, How about doing the following: * For '/browse' URL -- throw a 400 "Bad Request" (with a nicer looking error page), as it is missing a required parameter * For '/browse?type=invalidBrowseOption' URLs -- you throw a 404 "Not Found" (as the required parameter was included, but it was a value that didn't correspond to anything in the system) To me those responses would seem the most logical -- but I'm up for other ideas as well. > Interal System Error when browsing with wrong argument > ------------------------------------------------------ > > Key: DS-640 > URL: https://jira.duraspace.org/browse/DS-640 > Project: DSpace > Issue Type: Bug > Components: Documentation, JSPUI > Affects Versions: 1.5.2, 1.6.0, 1.6.1, 1.6.2 > Reporter: Hardik Mishra > Assignee: Stuart Lewis > Priority: Major > > On Browsing Items: > If someone tries to use browse type for which browse index does not exist, > like browse=publisher > e.g. http://dspace.webinito.com/browse?type > OR > e.g. http://dspace.webinito.com/browse?type=xyz > You will get Internal System Error -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.duraspace.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira ------------------------------------------------------------------------------ Nokia and AT&T present the 2010 Calling All Innovators-North America contest Create new apps & games for the Nokia N8 for consumers in U.S. and Canada $10 million total in prizes - $4M cash, 500 devices, nearly $6M in marketing Develop with Nokia Qt SDK, Web Runtime, or Java and Publish to Ovi Store http://p.sf.net/sfu/nokia-dev2dev _______________________________________________ Dspace-devel mailing list Dspace-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dspace-devel