Upgrade BeaconError to allow for 200 response code

ga4gh-beacon / specification

GA4GH Beacon specification.

Apache License 2.0

32 stars 25 forks source link

Upgrade BeaconError to allow for 200 response code #262

Open jrambla opened 5 years ago

jrambla commented 5 years ago

Currently, the description of BeaconError is "Beacon-specific error. This should be non-null in exceptional situations only, in which case exists has to be null." I believe that being precise in the values of attributes is desirable, given the ambiguous meaning of "null" ("no" vs "I don't know"). Thus I'll like to update the description to allow for a 200 response code, while keeping "null" also for retrocompatibility, but "not recommended".

jrambla commented 5 years ago

Beacon.md should be also updated to better inform on errors or return codes management.

sdelatorrep commented 5 years ago

I created PR #267