)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7973,"name":"Douglas Mendizábal","email":"dmendiza@redhat.com","username":"dougmendizabal"},"change_message_id":"d4d08eeeb3d2e0414df6d1d6aca3065c84159fb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6ebced0f_2166d6c7","updated":"2021-12-06 14:53:11.000000000","message":"Would it be possible to add a test to prevent a regression for this?","commit_id":"08ccfa98e86ff788070cb129bf7a35ca774cb811"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"839a87bf7efc0d49dc1d421152c9d4ee29e2beb3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"af313126_e9c54224","updated":"2021-11-22 21:18:17.000000000","message":"recheck","commit_id":"08ccfa98e86ff788070cb129bf7a35ca774cb811"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"a7b235cfb96e45c3e0ef4dbf7efcf69c104f8861","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ea2da6cd_83e074d9","updated":"2021-12-01 15:23:02.000000000","message":"recheck - unrelated doc failure ","commit_id":"08ccfa98e86ff788070cb129bf7a35ca774cb811"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"e851011fc349bbbe06425c07b7ce6dfea2cfe785","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e2ee5c31_64329fec","in_reply_to":"6ebced0f_2166d6c7","updated":"2022-02-04 15:45:16.000000000","message":"I changed the definition of the method to make all of the initialization values manatory. This should take care of any possible regressions.","commit_id":"08ccfa98e86ff788070cb129bf7a35ca774cb811"},{"author":{"_account_id":7414,"name":"David Wilde","email":"dwilde@redhat.com","username":"d34dh0r53"},"change_message_id":"031a46811cef7eb4e71eb3eec6e7e0853593eb2a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"017537ce_a0cb7318","updated":"2022-02-17 14:49:49.000000000","message":"recheck","commit_id":"d023b103e550f8237cb3cea72b99bbcf70791413"}],"keystone/credential/providers/fernet/core.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"71a0f0681ba6097e46790799b135f4db560617fb","unresolved":false,"context_lines":[{"line_number":96,"context_line":"        :param credential: an encrypted credential string"},{"line_number":97,"context_line":"        :returns: a decrypted credential"},{"line_number":98,"context_line":"        \"\"\""},{"line_number":99,"context_line":"        key_utils \u003d fernet_utils.FernetUtils("},{"line_number":100,"context_line":"            CONF.credential.key_repository, MAX_ACTIVE_KEYS, \u0027credential\u0027)"},{"line_number":101,"context_line":"        keys \u003d key_utils.load_keys(use_null_key\u003dTrue)"},{"line_number":102,"context_line":"        fernet_keys \u003d [fernet.Fernet(key) for key in keys]"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_462ec4fa","line":99,"range":{"start_line":99,"start_character":33,"end_line":99,"end_character":44},"updated":"2019-12-06 04:18:38.000000000","message":"Since credentials, tokens, and receipts all have configuration groups, we could add some level of validation into the FernetUtils constructor that requires all arguments at creation time.","commit_id":"08ccfa98e86ff788070cb129bf7a35ca774cb811"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"e851011fc349bbbe06425c07b7ce6dfea2cfe785","unresolved":false,"context_lines":[{"line_number":96,"context_line":"        :param credential: an encrypted credential string"},{"line_number":97,"context_line":"        :returns: a decrypted credential"},{"line_number":98,"context_line":"        \"\"\""},{"line_number":99,"context_line":"        key_utils \u003d fernet_utils.FernetUtils("},{"line_number":100,"context_line":"            CONF.credential.key_repository, MAX_ACTIVE_KEYS, \u0027credential\u0027)"},{"line_number":101,"context_line":"        keys \u003d key_utils.load_keys(use_null_key\u003dTrue)"},{"line_number":102,"context_line":"        fernet_keys \u003d [fernet.Fernet(key) for key in keys]"}],"source_content_type":"text/x-python","patch_set":1,"id":"10297984_45525901","line":99,"range":{"start_line":99,"start_character":33,"end_line":99,"end_character":44},"in_reply_to":"3fa7e38b_462ec4fa","updated":"2022-02-04 15:45:16.000000000","message":"I checked all other instantiations of FernetUtils and they all provide all of the arguments, so I\u0027m making the change.","commit_id":"08ccfa98e86ff788070cb129bf7a35ca774cb811"}]}
