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.

Sign up for GitHub

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

Jump to bottom

Perform option validation #18

Open
bertfrees opened this issue Sep 14, 2018 · 0 comments
Open

Perform option validation #18

bertfrees opened this issue Sep 14, 2018 · 0 comments
Labels
2 - In Progress enhancement

Comments

@bertfrees
Copy link
Member

bertfrees commented Sep 14, 2018 •
edited
Loading

Perform option validation before submitting a job request. Modules expect that options have the right values, otherwise unexpected things may happen. So option validation by the client is currently kind of part of the contract. There are two approaches:

A third approach is to make validation by the client optional, and always perform validation (again) in the framework (and return a HTTP 400 when validation failed).

Related: #19
bertfrees added a commit to daisy/pipeline that referenced this issue Nov 5, 2018
@bertfrees
[cli] Validate options using datatypes API
85b502b 
See daisy/pipeline-cli-go#18.

Also check that input files exist when in localfs mode.
bertfrees added a commit that referenced this issue Nov 30, 2018
@bertfrees
Validate options using datatypes API
cf5c855 
See #18.

Also check that input files exist when in localfs mode.
bertfrees added a commit that referenced this issue Feb 5, 2019
@bertfrees
Validate options using datatypes API
edc4920 
See #18.

Also check that input files exist when in localfs mode.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
2 - In Progress enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bertfrees