)]}'
{"monasca_log_api/tests/base.py":[{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"2afcdef5bf8e91aca423e9e2415784cfe2f91aa8","unresolved":false,"context_lines":[{"line_number":16,"context_line":"from oslo_config import fixture"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"def mock_config(test):"},{"line_number":20,"context_line":"    return fixture.Config()"}],"source_content_type":"text/x-python","patch_set":2,"id":"dae33548_c32ff49f","line":19,"range":{"start_line":19,"start_character":16,"end_line":19,"end_character":20},"updated":"2016-02-15 06:51:33.000000000","message":"If not needed please remove it.","commit_id":"ad984a6c50d11bc9635773e10706f1fa6409041b"}],"monasca_log_api/tests/test_log_publisher.py":[{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"2afcdef5bf8e91aca423e9e2415784cfe2f91aa8","unresolved":false,"context_lines":[{"line_number":107,"context_line":"class TestSendMessage(testtools.TestCase):"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    def setUp(self):"},{"line_number":110,"context_line":"        self.conf \u003d base.mock_config(self)"},{"line_number":111,"context_line":"        return super(TestSendMessage, self).setUp()"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"    @mock.patch(\u0027monasca_log_api.v2.common.log_publisher.producer\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"dae33548_a303d00e","line":110,"range":{"start_line":110,"start_character":20,"end_line":110,"end_character":42},"updated":"2016-02-15 06:51:33.000000000","message":"If you removed use fixture call from base test, you need to have it here.","commit_id":"ad984a6c50d11bc9635773e10706f1fa6409041b"}],"monasca_log_api/v2/common/log_publisher.py":[{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"2afcdef5bf8e91aca423e9e2415784cfe2f91aa8","unresolved":false,"context_lines":[{"line_number":170,"context_line":"            for topic in self._topics:"},{"line_number":171,"context_line":"                self._publisher().publish(topic, msg, key)"},{"line_number":172,"context_line":"        except KeyError as ex:"},{"line_number":173,"context_line":"            self._kafka_publisher \u003d None"},{"line_number":174,"context_line":"            LOG.error(ex.message)"},{"line_number":175,"context_line":"            raise"},{"line_number":176,"context_line":"        except Exception as ex:"}],"source_content_type":"text/x-python","patch_set":2,"id":"dae33548_9e1a636d","line":173,"range":{"start_line":173,"start_character":17,"end_line":173,"end_character":33},"updated":"2016-02-15 06:51:33.000000000","message":"I think that closing connections should also be done here if possible.","commit_id":"ad984a6c50d11bc9635773e10706f1fa6409041b"}],"monasca_log_api/v2/reference/logs.py":[{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"2afcdef5bf8e91aca423e9e2415784cfe2f91aa8","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        cross_tenant_id \u003d self.json_args.get(\u0027tenant_id\u0027, None)"},{"line_number":53,"context_line":"        tenant_id \u003d self.request.headers.get(headers.X_TENANT_ID.name, \u0027\u0027)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        self._before_logs_post(cross_tenant_id, tenant_id)"},{"line_number":56,"context_line":"        service.Validations.validate_payload_size(self.request)"},{"line_number":57,"context_line":"        service.Validations.validate_content_type(self.request)"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"        log \u003d self.get_log(req\u003dself.request)"},{"line_number":60,"context_line":"        envelope \u003d self.get_envelope("}],"source_content_type":"text/x-python","patch_set":2,"id":"dae33548_beb92731","line":57,"range":{"start_line":55,"start_character":7,"end_line":57,"end_character":63},"updated":"2016-02-15 06:51:33.000000000","message":"I think that three calls should happen in \n\n prepare\n\nI assume that this is for single log. Therefore I find it essential to validate request prior to any deserialization would happen.","commit_id":"ad984a6c50d11bc9635773e10706f1fa6409041b"}]}
