diff options
author | Romain Forlot <romain.forlot@iot.bzh> | 2018-09-16 02:38:26 +0200 |
---|---|---|
committer | Romain Forlot <romain.forlot@iot.bzh> | 2018-09-27 13:05:37 +0200 |
commit | 7138e18c162f9246079fbb077d38f7796314b6b0 (patch) | |
tree | 0d76110c2f04c0c0ba4ea310b2318390aea3b330 /signal-composer-binding/signal-composer-binding.cpp | |
parent | e0655ca2d3d2db4c99d56da6fc90345e9093deb7 (diff) |
Handle errors on signal initialization
This reports errors at signals getSignals action call
and removed incorrect signals from the source. This allows
the binding to correctly respond to the user if there was an
error in its signal definitions.
Change-Id: I7779ab9339a979fcb2b7b41f8ebf4aa27b8af3a1
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
Diffstat (limited to 'signal-composer-binding/signal-composer-binding.cpp')
-rw-r--r-- | signal-composer-binding/signal-composer-binding.cpp | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/signal-composer-binding/signal-composer-binding.cpp b/signal-composer-binding/signal-composer-binding.cpp index 5932640..310cb40 100644 --- a/signal-composer-binding/signal-composer-binding.cpp +++ b/signal-composer-binding/signal-composer-binding.cpp @@ -211,13 +211,19 @@ void addObjects(afb_req request) if(sourcesJ && composer.loadSources(sourcesJ)) { - afb_req_fail_f(request, "Loading 'sources' configuration or subscription error", "Error code: -2"); + afb_req_fail_f(request, "Loading 'sources' configuration or subscription error", "Error code: -4"); return; } if(signalsJ) { if(!composer.loadSignals(signalsJ)) - {composer.initSignals();} + { + if(composer.initSignals()) + { + afb_req_fail_f(request, "Loading 'signals' configuration or subscription error", "Error code: -3"); + return; + } + } else { afb_req_fail_f(request, "Loading 'signals' configuration or subscription error", "Error code: -2"); |