Improved servlet lifecycles for AudioServlet and ChartServlet #6161
Conversation
Signed-off-by: Kai Kreuzer <kai@openhab.org>
Signed-off-by: Kai Kreuzer <kai@openhab.org>
This pull request has been mentioned on openHAB Community. There might be relevant details there: https://community.openhab.org/t/double-chartservlet-in-http-list-charts-error/50919/3 |
Signed-off-by: Kai Kreuzer <kai@openhab.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, one small cosmetic inline.
@@ -125,9 +124,6 @@ protected void activate() { | |||
protected void deactivate() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could be removed.
Signed-off-by: Kai Kreuzer <kai@openhab.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks.
Signed-off-by: Kai Kreuzer <kai@openhab.org>
This pull request has been mentioned on openHAB Community. There might be relevant details there: https://community.openhab.org/t/double-chartservlet-in-http-list-charts-error/50919/9 |
Signed-off-by: Kai Kreuzer <kai@openhab.org>
…e-archived#6161) * Improved servlet lifecycles for AudioServlet and ChartServlet * also adapted DefaultChartProvider * send proper HTTPResponses instead of exceptions * removed empty deactivate method * properly handle NumberFormatExceptions * changed config description to correct types Signed-off-by: Kai Kreuzer <kai@openhab.org>
Signed-off-by: Kai Kreuzer kai@openhab.org