fiware-cygnus
fiware-cygnus copied to clipboard
[cygnus-ngsi][NGSIRestHandler] Do not throw exceptions, just trace them
NGSIRestHandler
traces and throws the following exceptions:
They must be just traced.
@frbattid As per my understanding, the fix of this issue is to put error logs instead of throw exception at the mentioned line number of NGSIRestHandler.java. But at line number 233 of NGSIRestHandler.java, HTTPBadRequestException is thrown if "/" is not added in servicepath. So if we fix the issue then in this case the error log will be logged but code will execute successfully with incorrect servicepath.
Please confirm my understanding
I understand that the work consists in properly trace exception using the logging library instead of getting a stack trace but that the code flow will be as it is now (I mean if the execution flow is not interrupted due to an incorrect service path that shouldn't change).
Just guessing. Honestly, I'm not sure about what @frbattid has in mind regarding this :)... (and I'm afraid he is no longer active on Cygnus)