)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7973,"name":"Douglas Mendizábal","email":"dmendiza@redhat.com","username":"dougmendizabal"},"change_message_id":"284093217adc5034168d9235105eb46c512ff6d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fb6460cb_6272a8fe","updated":"2025-02-27 15:31:30.000000000","message":"recheck - timeout","commit_id":"fc6bfe1e8adaa55eddf39461d1f43232215e1aeb"},{"author":{"_account_id":9914,"name":"Ade Lee","email":"alee@redhat.com","username":"alee"},"change_message_id":"95380fc8bb589b3710f448ad084ecce5a0d57265","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c1ead54d_f0cbbc5a","updated":"2025-02-27 15:42:50.000000000","message":"small changes","commit_id":"fc6bfe1e8adaa55eddf39461d1f43232215e1aeb"}],"barbican/plugin/crypto/simple_crypto.py":[{"author":{"_account_id":9914,"name":"Ade Lee","email":"alee@redhat.com","username":"alee"},"change_message_id":"95380fc8bb589b3710f448ad084ecce5a0d57265","unresolved":true,"context_lines":[{"line_number":59,"context_line":"        if len(conf.simple_crypto_plugin.kek) \u003c 1:"},{"line_number":60,"context_line":"            raise ValueError(u._(\"SimpleCrypto KEK is undefined\"))"},{"line_number":61,"context_line":"        self.master_keys \u003d conf.simple_crypto_plugin.kek"},{"line_number":62,"context_line":"        self.master_kek \u003d self.master_keys[0]"},{"line_number":63,"context_line":"        self.plugin_name \u003d conf.simple_crypto_plugin.plugin_name"},{"line_number":64,"context_line":"        LOG.info(\"{} initialized\".format(self.plugin_name))"},{"line_number":65,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3f607d10_0b2175c6","line":62,"range":{"start_line":62,"start_character":8,"end_line":62,"end_character":45},"updated":"2025-02-27 15:42:50.000000000","message":"This field is redundant now.  I think we should remove it to avoid confusion.","commit_id":"fc6bfe1e8adaa55eddf39461d1f43232215e1aeb"},{"author":{"_account_id":7973,"name":"Douglas Mendizábal","email":"dmendiza@redhat.com","username":"dougmendizabal"},"change_message_id":"00831452bd24177e29f437860c84157d5d24087c","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        if len(conf.simple_crypto_plugin.kek) \u003c 1:"},{"line_number":60,"context_line":"            raise ValueError(u._(\"SimpleCrypto KEK is undefined\"))"},{"line_number":61,"context_line":"        self.master_keys \u003d conf.simple_crypto_plugin.kek"},{"line_number":62,"context_line":"        self.master_kek \u003d self.master_keys[0]"},{"line_number":63,"context_line":"        self.plugin_name \u003d conf.simple_crypto_plugin.plugin_name"},{"line_number":64,"context_line":"        LOG.info(\"{} initialized\".format(self.plugin_name))"},{"line_number":65,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"156e2251_e70349bd","line":62,"range":{"start_line":62,"start_character":8,"end_line":62,"end_character":45},"in_reply_to":"3f607d10_0b2175c6","updated":"2025-02-27 16:25:34.000000000","message":"Done","commit_id":"fc6bfe1e8adaa55eddf39461d1f43232215e1aeb"},{"author":{"_account_id":9914,"name":"Ade Lee","email":"alee@redhat.com","username":"alee"},"change_message_id":"95380fc8bb589b3710f448ad084ecce5a0d57265","unresolved":true,"context_lines":[{"line_number":108,"context_line":"        kek_meta_dto.mode \u003d \u0027cbc\u0027"},{"line_number":109,"context_line":"        if not kek_meta_dto.plugin_meta:"},{"line_number":110,"context_line":"            # the kek is stored encrypted in the plugin_meta field"},{"line_number":111,"context_line":"            encryptor \u003d fernet.Fernet(self.master_kek)"},{"line_number":112,"context_line":"            key \u003d fernet.Fernet.generate_key()"},{"line_number":113,"context_line":"            kek_meta_dto.plugin_meta \u003d encryptor.encrypt(key)"},{"line_number":114,"context_line":"        return kek_meta_dto"}],"source_content_type":"text/x-python","patch_set":2,"id":"556f9415_343a2922","line":111,"range":{"start_line":111,"start_character":38,"end_line":111,"end_character":53},"updated":"2025-02-27 15:42:50.000000000","message":"just use self.master_keys[0]","commit_id":"fc6bfe1e8adaa55eddf39461d1f43232215e1aeb"},{"author":{"_account_id":7973,"name":"Douglas Mendizábal","email":"dmendiza@redhat.com","username":"dougmendizabal"},"change_message_id":"00831452bd24177e29f437860c84157d5d24087c","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        kek_meta_dto.mode \u003d \u0027cbc\u0027"},{"line_number":109,"context_line":"        if not kek_meta_dto.plugin_meta:"},{"line_number":110,"context_line":"            # the kek is stored encrypted in the plugin_meta field"},{"line_number":111,"context_line":"            encryptor \u003d fernet.Fernet(self.master_kek)"},{"line_number":112,"context_line":"            key \u003d fernet.Fernet.generate_key()"},{"line_number":113,"context_line":"            kek_meta_dto.plugin_meta \u003d encryptor.encrypt(key)"},{"line_number":114,"context_line":"        return kek_meta_dto"}],"source_content_type":"text/x-python","patch_set":2,"id":"4a8aa27d_e0a37c1b","line":111,"range":{"start_line":111,"start_character":38,"end_line":111,"end_character":53},"in_reply_to":"556f9415_343a2922","updated":"2025-02-27 16:25:34.000000000","message":"Done","commit_id":"fc6bfe1e8adaa55eddf39461d1f43232215e1aeb"}]}
