)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"3c16bc1235b39d13b157a6db49ea4b93040d064e","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Tim Burke \u003ctim.burke@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-01-05 12:28:33 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"s3api: Ensure parsed ACLs are not None"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #1999564"},{"line_number":10,"context_line":"Change-Id: Ia7bbefa9df757d91a4364704d83207aeb1db648a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"592ae988_cb2d5402","line":7,"updated":"2023-01-13 14:04:52.000000000","message":"nit: the ACL was never None, it\u0027s elements of the ACL that cannot be None","commit_id":"72d23e956280161149b72c8e520f0f329fa83073"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"3c16bc1235b39d13b157a6db49ea4b93040d064e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d6160201_f1bea181","updated":"2023-01-13 14:04:52.000000000","message":"I\u0027m concerned about defaulting an empty Grantee ID to \"None\".\n\nAFAICT, only one of the changes is covered by the test.","commit_id":"72d23e956280161149b72c8e520f0f329fa83073"}],"swift/common/middleware/s3api/subresource.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"3c16bc1235b39d13b157a6db49ea4b93040d064e","unresolved":true,"context_lines":[{"line_number":166,"context_line":"    def from_elem(elem):"},{"line_number":167,"context_line":"        type \u003d elem.get(\u0027{%s}type\u0027 % XMLNS_XSI)"},{"line_number":168,"context_line":"        if type \u003d\u003d \u0027CanonicalUser\u0027:"},{"line_number":169,"context_line":"            value \u003d str(elem.find(\u0027./ID\u0027).text)"},{"line_number":170,"context_line":"            return User(value)"},{"line_number":171,"context_line":"        elif type \u003d\u003d \u0027Group\u0027:"},{"line_number":172,"context_line":"            value \u003d str(elem.find(\u0027./URI\u0027).text)"}],"source_content_type":"text/x-python","patch_set":1,"id":"88562af3_1240ed7d","line":169,"updated":"2023-01-13 14:04:52.000000000","message":"if test is None, does this end up granting to a CanonicalUser named \"None\"?","commit_id":"72d23e956280161149b72c8e520f0f329fa83073"}]}