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

As a user, I want to get a bad request error if I use a URL parameter which is not supported #252

Open
tloubrieu-jpl opened this issue Jan 23, 2023 · 1 comment
Assignees
Labels
needs:triage requirement the current issue is a requirement

Comments

@tloubrieu-jpl
Copy link
Member

Checked for duplicates

No - I haven't checked

Related problems

I'm frustrated when make a typo in a parameter and I get a result from the API anyway but without my parameter being understood.

πŸ§‘β€πŸ”¬ User Persona(s)

Data user

πŸ’ͺ Motivation

...so that I can quickly spot wrong usage of the API

πŸ“– Additional Details

Check tht there is not a extensibility motivation for supporting extra parameter silently. However I believe this is true at the specification level (the specification is extensible) but should not be true at the implementation level (the implementation is limited and should be explicit about it)

Acceptance Criteria

Given deployed server
When I perform a request curl http://{server-host}/product?keyword=thing
Then I expect i expect an error 400, with a text message saying parameter 'keyword' is not supported by the API

βš™οΈ Engineering Details

Provide some design / implementation details and/or a sub-task checklist as needed. Convert issue to Epic if estimate is outside the scope of 1 sprint.

@tloubrieu-jpl tloubrieu-jpl added requirement the current issue is a requirement needs:triage labels Jan 23, 2023
@tloubrieu-jpl
Copy link
Member Author

The feature request is motivated by ticket NASA-PDS/registry-api#227

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs:triage requirement the current issue is a requirement
Projects
Status: ToDo
Development

No branches or pull requests

2 participants