)]}'
{"glance/api/v2/metadef_namespaces.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"8af1a39bf40eaf9f8b229074c0e48e52d5d538b3","unresolved":true,"context_lines":[{"line_number":269,"context_line":"                    md_resource\u003dnamespace_obj,"},{"line_number":270,"context_line":"                    enforcer\u003dself.policy)"},{"line_number":271,"context_line":"                policy_check.get_metadef_namespace()"},{"line_number":272,"context_line":"            except (exception.Forbidden, webob.exc.HTTPForbidden):"},{"line_number":273,"context_line":"                LOG.debug(\"User not permitted to show namespace \u0027%s\u0027\","},{"line_number":274,"context_line":"                          namespace)"},{"line_number":275,"context_line":"                # NOTE (abhishekk): Returning 404 Not Found as the"}],"source_content_type":"text/x-python","patch_set":2,"id":"f9a73daa_f324ede0","line":272,"updated":"2021-08-24 13:44:36.000000000","message":"This is a good reason to not doubly-nest this stuff inside the giant anything-goes exception handler, IMHO. If we did this above before L261, then this would have been a 500 and we would have realized earlier.","commit_id":"f33504c39b5e829c7dccae133ea63017dd942999"}],"glance/tests/unit/v2/test_metadef_resources.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"8af1a39bf40eaf9f8b229074c0e48e52d5d538b3","unresolved":true,"context_lines":[{"line_number":451,"context_line":""},{"line_number":452,"context_line":"    def test_namespace_show_non_visible(self):"},{"line_number":453,"context_line":"        request \u003d unit_test_utils.get_fake_request()"},{"line_number":454,"context_line":"        self.assertRaises(webob.exc.HTTPForbidden,"},{"line_number":455,"context_line":"                          self.namespace_controller.show, request, NAMESPACE2)"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":"    def test_namespace_delete(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"fecb86be_6b2b4f78","side":"PARENT","line":454,"updated":"2021-08-24 13:44:36.000000000","message":"I guess this was missed on review. Seems obvious now :)","commit_id":"cb3369002ade102508bf9c0f776ed7377751789e"}]}