)]}'
{"keystonemiddleware/auth_token/__init__.py":[{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"3534b595e9cdb60c37be6a6aef93fb931998c93b","unresolved":true,"context_lines":[{"line_number":391,"context_line":"                        # Mark it so the rejection logic returns 403 rather"},{"line_number":392,"context_line":"                        # than 401."},{"line_number":393,"context_line":"                        request._service_token_auth_failed \u003d True"},{"line_number":394,"context_line":"                        request.service_token_valid \u003d False"},{"line_number":395,"context_line":"                        self.log.info("},{"line_number":396,"context_line":"                            \"The service token did not have the required roles\""},{"line_number":397,"context_line":"                        )"}],"source_content_type":"text/x-python","patch_set":3,"id":"137a0666_60b3131e","line":394,"updated":"2026-07-18 05:29:39.000000000","message":"This changes the behavior, should be split out to another patch","commit_id":"4792b2a945af1291a7143f93381cf52329a8464c"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"cc6bffe55dcadfa58bd163d92b5bf23396191916","unresolved":true,"context_lines":[{"line_number":391,"context_line":"                        # Mark it so the rejection logic returns 403 rather"},{"line_number":392,"context_line":"                        # than 401."},{"line_number":393,"context_line":"                        request._service_token_auth_failed \u003d True"},{"line_number":394,"context_line":"                        request.service_token_valid \u003d False"},{"line_number":395,"context_line":"                        self.log.info("},{"line_number":396,"context_line":"                            \"The service token did not have the required roles\""},{"line_number":397,"context_line":"                        )"}],"source_content_type":"text/x-python","patch_set":3,"id":"d3f735e1_93b13554","line":394,"in_reply_to":"137a0666_60b3131e","updated":"2026-07-20 08:56:59.000000000","message":"you are very right, the change was initially created on top of https://review.opendev.org/c/openstack/keystonemiddleware/+/989646 and after I rebased it somehow the change was merged here - it is not visible on it\u0027s own in the base-PS1 diff. I have recreated everything from scratch in the following way:\n- clean master branch\n- add .pre-commit\n- run `pre-commit run -a`\n- fix 2 issues manually (renamed `l` to `_l`): \n```\nkeystonemiddleware/tests/unit/test_opts.py:72:37: E741 Ambiguous variable name: `l`\n   |\n70 |             \u0027service_type\u0027,\n71 |         ]\n72 |         opt_names \u003d [o.name for (g, l) in result_of_old_opts for o in l]\n   |                                     ^ E741\n73 |         self.assertThat(opt_names, matchers.HasLength(len(expected_opt_names)))\n   |\n\nkeystonemiddleware/tests/unit/test_opts.py:124:37: E741 Ambiguous variable name: `l`\n    |\n122 |             \u0027service_type\u0027,\n123 |         ]\n124 |         opt_names \u003d [o.name for (g, l) in result for o in l]\n    |                                     ^ E741\n125 |         self.assertThat(opt_names, matchers.HasLength(len(expected_opt_names)))\n```\n- commit","commit_id":"4792b2a945af1291a7143f93381cf52329a8464c"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"3534b595e9cdb60c37be6a6aef93fb931998c93b","unresolved":true,"context_lines":[{"line_number":708,"context_line":"                            \"message\": message,"},{"line_number":709,"context_line":"                        }"},{"line_number":710,"context_line":"                    }"},{"line_number":711,"context_line":"                    raise webob.exc.HTTPForbidden("},{"line_number":712,"context_line":"                        body\u003djsonutils.dumps(body),"},{"line_number":713,"context_line":"                        charset\u003d\"UTF-8\","},{"line_number":714,"context_line":"                        content_type\u003d\"application/json\","}],"source_content_type":"text/x-python","patch_set":3,"id":"366e44ff_f18d8e14","line":711,"updated":"2026-07-18 05:29:39.000000000","message":"same - change of behavior","commit_id":"4792b2a945af1291a7143f93381cf52329a8464c"}]}