Ticket #1195 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Storage extension to handle incorrect routes better

Reported by: dread Owned by: rgrp
Priority: minor Milestone: ckan-sprint-2011-10-28
Component: ckan Keywords:
Cc: Repository: ckanext-storage
Theme: none

Description

We've seen exceptions caused by people getting the file storage link wrong, adding a slash to the end.

e.g. ​http://ckan.net/storage/f/file/e696d857-e997-41c8-be08-408697168ca8/

We should aim to handle all requests better than causing exception. 404 (or even 302 in this case) is better than 500.

Change History

comment:1 Changed 3 years ago by dread

Also this one: ​http://ckan.net/storage/f/file/

comment:2 Changed 3 years ago by rgrp

  • Repository changed from ckan to ckanext-storage
  • Milestone set to ckan-current-sprint

Fixed in ​https://bitbucket.org/okfn/ckanext-storage/changeset/49734285f1ef

comment:3 Changed 3 years ago by rgrp

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.