)]}'
{"keystone/tests/unit/server/test_keystone_flask.py":[{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"c2ededc4e5008a53096146757c3e9280b3acef6b","unresolved":true,"context_lines":[{"line_number":858,"context_line":"        self, mock_log_exc, mock_log_debug"},{"line_number":859,"context_line":"    ):"},{"line_number":860,"context_line":"        \"\"\"Test that ValidationError (400) is logged at DEBUG.\"\"\""},{"line_number":861,"context_line":"        error \u003d exception.ValidationError(attribute\u003d\u0027name\u0027, target\u003d\u0027resource\u0027)"},{"line_number":862,"context_line":"        with self.test_client() as c:"},{"line_number":863,"context_line":"            with self.public_app.app.test_request_context(\u0027/\u0027):"},{"line_number":864,"context_line":"                flask_application._handle_keystone_exception(error)"}],"source_content_type":"text/x-python","patch_set":1,"id":"4735f6e2_0624ea2f","line":861,"updated":"2026-03-20 15:50:01.000000000","message":"Right now this exception is the only way to figure out what exactly is wrong in the request or whether the schema itself is wrong. I remember I had some cases where only the whole exception was able to give me insight into what exactly was wrong. I generally agree for everything \u003c500 we should not log exceptions, but exceptions are the only way to get stacktraces to be able to catch the issue. I am worrying we loose some insides with this change","commit_id":"0c27c5750e1125f78767e920c247540ca854a001"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"485358f6547e57e022b9ebc1f223081da05e8c19","unresolved":true,"context_lines":[{"line_number":858,"context_line":"        self, mock_log_exc, mock_log_debug"},{"line_number":859,"context_line":"    ):"},{"line_number":860,"context_line":"        \"\"\"Test that ValidationError (400) is logged at DEBUG.\"\"\""},{"line_number":861,"context_line":"        error \u003d exception.ValidationError(attribute\u003d\u0027name\u0027, target\u003d\u0027resource\u0027)"},{"line_number":862,"context_line":"        with self.test_client() as c:"},{"line_number":863,"context_line":"            with self.public_app.app.test_request_context(\u0027/\u0027):"},{"line_number":864,"context_line":"                flask_application._handle_keystone_exception(error)"}],"source_content_type":"text/x-python","patch_set":1,"id":"86383ce2_e4958ebd","line":861,"in_reply_to":"4735f6e2_0624ea2f","updated":"2026-03-20 15:53:37.000000000","message":"What we could do is log an exception when debug\u003dtrue, and leave it as a debug message when debug\u003dfalse. Would it work for you?","commit_id":"0c27c5750e1125f78767e920c247540ca854a001"}]}
