Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

redirect /collection/AllCollectionReport to /collection/AllCollectionReport.html #261

Open
wowasa opened this issue Sep 29, 2024 · 1 comment
Assignees

Comments

@wowasa
Copy link
Collaborator

wowasa commented Sep 29, 2024

an explicit requirement from our September 25th meeting was to redirect a call to /collection/AllCollectionReport to /collection/AllCollectionReport.html. I make it more general:

  • any html report should be callable with and without .html suffix
  • if a report doesn't exist under the given name, the response must return a status code 404 instead of 500
@wowasa wowasa self-assigned this Sep 29, 2024
@twagoo
Copy link
Member

twagoo commented Sep 30, 2024

For the record, redirect /collection/AllCollectionReport to /collection/AllCollectionReport.html IFF HTML is requested via the Accept header. In the case of JSON, XML, TSV I think the response should be in the requested format without a redirect.

wowasa added a commit that referenced this issue Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants