FlexMeasures / flexmeasures / 23710691614

78%

push

github

Feat/handle security error (#2054)

* feat: security error handler

Signed-off-by: F.N. Claessen <claessen@seita.nl>

* refactor: reuse more robust code from generic error_handling_router

Signed-off-by: F.N. Claessen <claessen@seita.nl>

* feat: prefer less verbose log statement for SecurityError

Signed-off-by: F.N. Claessen <claessen@seita.nl>

* feat: still log the error description, even if not verbose (only exclude the stack trace)

Signed-off-by: F.N. Claessen <claessen@seita.nl>

* style: add space in case of fallback

Signed-off-by: F.N. Claessen <claessen@seita.nl>

* feat: improve logging by also logging the error description for 401, 403 and 410 errors (only the 404 description is pointless to log)

Signed-off-by: F.N. Claessen <claessen@seita.nl>

* docs: remove obsolete comment

Signed-off-by: F.N. Claessen <claessen@seita.nl>

* docs: changelog entry

Signed-off-by: F.N. Claessen <claessen@seita.nl>

* refactor: all listed Exceptions are subclasses of Exception

Signed-off-by: F.N. Claessen <claessen@seita.nl>

* style: flake8 (remove obsolete imports)

Signed-off-by: F.N. Claessen <claessen@seita.nl>

---------

Signed-off-by: F.N. Claessen <claessen@seita.nl>
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Co-authored-by: Nicolas Höning <nicolas@seita.nl>

6 of 7 new or added lines in 1 file covered. (85.71%)

11455 of 14753 relevant lines covered (77.65%)

0.78 hits per line