)]}'
{"barbican/common/validators.py":[{"author":{"_account_id":9914,"name":"Ade Lee","email":"alee@redhat.com","username":"alee"},"change_message_id":"412ae23146a89fe4be504cbfec2af6cfb2affecc","unresolved":true,"context_lines":[{"line_number":535,"context_line":"        bit_length \u003d int(meta.get(\u0027bit_length\u0027))"},{"line_number":536,"context_line":"        algorithm \u003d meta.get(\u0027algorithm\u0027, \u0027\u0027).lower()"},{"line_number":537,"context_line":"        if bit_length % 8 !\u003d 0:"},{"line_number":538,"context_line":"            if algorithm \u003d\u003d \u0027ec\u0027 and bit_length in self._EC_BIT_LENGTHS:"},{"line_number":539,"context_line":"                return"},{"line_number":540,"context_line":"            raise exception.UnsupportedField(field\u003d\"bit_length\","},{"line_number":541,"context_line":"                                             schema\u003dschema_name,"},{"line_number":542,"context_line":"                                             reason\u003du._(\"Must be a\""}],"source_content_type":"text/x-python","patch_set":1,"id":"32ecbb38_bebc3e60","line":539,"range":{"start_line":538,"start_character":11,"end_line":539,"end_character":22},"updated":"2026-05-29 14:40:58.000000000","message":"This check is in the wrong place.  You should do this first.  For example, 256 is a multiple of 8 and it passes, but just because its a multiple of 8 and doesn\u0027t reach this code.","commit_id":"3ca0b680e5bcc7239f24c250b92ebadfb7472474"},{"author":{"_account_id":32946,"name":"Joshua Cornutt","email":"joshua@open-edge.io","username":"jcornutt"},"change_message_id":"71b34a57884a4439975eb30a7fb46c390c4f9c63","unresolved":true,"context_lines":[{"line_number":535,"context_line":"        bit_length \u003d int(meta.get(\u0027bit_length\u0027))"},{"line_number":536,"context_line":"        algorithm \u003d meta.get(\u0027algorithm\u0027, \u0027\u0027).lower()"},{"line_number":537,"context_line":"        if bit_length % 8 !\u003d 0:"},{"line_number":538,"context_line":"            if algorithm \u003d\u003d \u0027ec\u0027 and bit_length in self._EC_BIT_LENGTHS:"},{"line_number":539,"context_line":"                return"},{"line_number":540,"context_line":"            raise exception.UnsupportedField(field\u003d\"bit_length\","},{"line_number":541,"context_line":"                                             schema\u003dschema_name,"},{"line_number":542,"context_line":"                                             reason\u003du._(\"Must be a\""}],"source_content_type":"text/x-python","patch_set":1,"id":"5347f802_eba1ddd0","line":539,"range":{"start_line":538,"start_character":11,"end_line":539,"end_character":22},"in_reply_to":"32ecbb38_bebc3e60","updated":"2026-06-03 19:42:05.000000000","message":"Agreed. Reordered so the EC curve set is validated first. Also added a test for the multiple-of-8-but-not-a-curve case. PS2.","commit_id":"3ca0b680e5bcc7239f24c250b92ebadfb7472474"}]}