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

unrecognized custom parameter and optional parameters responses #17

Open
molinaro-m opened this issue Jul 24, 2020 · 1 comment
Open
Labels
1.1 bug Something isn't working

Comments

@molinaro-m
Copy link
Member

ConeSearch states (end of §2.1)

If a query includes an optional parameter, either one specified by this document or not, that is not supported by the service implementation, the service must ignore that parameter.

This looks like silent failure.
Maybe we can suggest (a SHOULD?) to add <INFO> elements reporting the unused parameters.

@Zarquan
Copy link
Member

Zarquan commented Aug 10, 2020

Quietly ignoring an unknown parameter and not reporting it is a recipe for a really bad end user experience. What if it is a known parameter but the user misspills it ? If this query is part of a larger workflow it could take hours to track it down.

Good software should fail early and fail loudly.

@molinaro-m molinaro-m added bug Something isn't working time labels Aug 27, 2020
@molinaro-m molinaro-m added 1.1 and removed time labels Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.1 bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants