search

clamsproject / wfe-pipeline-runner

3 stars 1 forks source link

Error handling #10

Closed marcverhagen closed 2 years ago

marcverhagen commented 3 years ago

Need much better error handling. At least return JSON instead of HTML

Need to think this through because unlike with LAPPS we do not guarantee we return a JSON object. In LAPPS, that JSON would have a type and a payload (where the type can be LIF or ERROR or whatnot).

GBH prefers fail fast rather than graceful degradation.

Use of error logs may be an option.

marcverhagen commented 3 years ago

Some of this was done in https://github.com/clamsproject/pipeline-runner/commit/d5794a93ee8baf26963dfd77ee4fdf9b6009dbfa. Still keeping this open because the current pipeline error handling needs to be updated once clams-python error handling is released.