)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"78d8a1b8e8df7a7af94bc2167ab4fea987f34af6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6d2124a9_cf0d59d9","updated":"2026-10-13 06:25:10.000000000","message":"@kajinamit@oss.nttdata.com thanks for your review. who else could look at this?","commit_id":"c28b400fdce69fed7a23314456caf230e3e676a9"}],"oslo_middleware/tests/test_auth_basic.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a7b51c1649b2635111ebba849693ef8b026244b5","unresolved":true,"context_lines":[{"line_number":283,"context_line":"        self.assertTrue(opt.deprecated_since)"},{"line_number":284,"context_line":""},{"line_number":285,"context_line":""},{"line_number":286,"context_line":"class TestAuthBasicOptionalBcrypt(test_base.BaseTestCase):"},{"line_number":287,"context_line":"    def setUp(self):"},{"line_number":288,"context_line":"        super().setUp()"},{"line_number":289,"context_line":"        with tempfile.NamedTemporaryFile(mode\u003d\u0027w\u0027, delete\u003dFalse) as f:"}],"source_content_type":"text/x-python","patch_set":2,"id":"a1f69a9d_a568dcbe","line":286,"range":{"start_line":286,"start_character":6,"end_line":286,"end_character":33},"updated":"2026-10-05 17:42:58.000000000","message":"Please try adding this to the existing test class instead.","commit_id":"1da04569682a8019142cb4a226053d159b8e65cc"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"622273a9bcd7d90e3338cf95bd5926f41e279f65","unresolved":false,"context_lines":[{"line_number":283,"context_line":"        self.assertTrue(opt.deprecated_since)"},{"line_number":284,"context_line":""},{"line_number":285,"context_line":""},{"line_number":286,"context_line":"class TestAuthBasicOptionalBcrypt(test_base.BaseTestCase):"},{"line_number":287,"context_line":"    def setUp(self):"},{"line_number":288,"context_line":"        super().setUp()"},{"line_number":289,"context_line":"        with tempfile.NamedTemporaryFile(mode\u003d\u0027w\u0027, delete\u003dFalse) as f:"}],"source_content_type":"text/x-python","patch_set":2,"id":"c1b02676_d4a081ce","line":286,"in_reply_to":"a1f69a9d_a568dcbe","updated":"2026-10-06 08:14:41.000000000","message":"Done, test_middleware_without_bcrypt is now in TestAuthBasic.","commit_id":"1da04569682a8019142cb4a226053d159b8e65cc"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a7b51c1649b2635111ebba849693ef8b026244b5","unresolved":true,"context_lines":[{"line_number":302,"context_line":""},{"line_number":303,"context_line":"        self.fake_app \u003d fake_app"},{"line_number":304,"context_line":""},{"line_number":305,"context_line":"    def test_import_does_not_load_bcrypt(self):"},{"line_number":306,"context_line":"        \"\"\"Consumers that merely import the package must not load bcrypt."},{"line_number":307,"context_line":""},{"line_number":308,"context_line":"        Check in a subprocess so that bcrypt already imported by other tests"}],"source_content_type":"text/x-python","patch_set":2,"id":"a10dfd01_f30535b0","line":305,"range":{"start_line":305,"start_character":8,"end_line":305,"end_character":40},"updated":"2026-10-05 17:42:58.000000000","message":"We don\u0027t usually implement such test case and it can be broken by dependencies. I would prefer removing it now.","commit_id":"1da04569682a8019142cb4a226053d159b8e65cc"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"622273a9bcd7d90e3338cf95bd5926f41e279f65","unresolved":false,"context_lines":[{"line_number":302,"context_line":""},{"line_number":303,"context_line":"        self.fake_app \u003d fake_app"},{"line_number":304,"context_line":""},{"line_number":305,"context_line":"    def test_import_does_not_load_bcrypt(self):"},{"line_number":306,"context_line":"        \"\"\"Consumers that merely import the package must not load bcrypt."},{"line_number":307,"context_line":""},{"line_number":308,"context_line":"        Check in a subprocess so that bcrypt already imported by other tests"}],"source_content_type":"text/x-python","patch_set":2,"id":"0296c77f_0f7bd3d3","line":305,"in_reply_to":"a10dfd01_f30535b0","updated":"2026-10-06 08:14:41.000000000","message":"Removed.","commit_id":"1da04569682a8019142cb4a226053d159b8e65cc"}],"pyproject.toml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"fa9c15462c94366daf87b0fc21a458c98a4bcfc0","unresolved":true,"context_lines":[{"line_number":30,"context_line":"dynamic \u003d [\"version\", \"dependencies\"]"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"[project.optional-dependencies]"},{"line_number":33,"context_line":"bcrypt \u003d ["},{"line_number":34,"context_line":"    \"bcrypt\u003e\u003d3.1.3\","},{"line_number":35,"context_line":"]"},{"line_number":36,"context_line":"tracing-http \u003d ["}],"source_content_type":"text/x-toml","patch_set":1,"id":"4249c328_8559eaf6","line":33,"range":{"start_line":33,"start_character":0,"end_line":33,"end_character":6},"updated":"2026-10-05 16:23:11.000000000","message":"Can we name the group \u0027basic-auth\u0027 instead so that we can understand what requires this.","commit_id":"cb70df73c6cb5924f8c373a0a298fc32bc62913d"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"476f67bf5b281962e4c3236d3607e4185ccfd75c","unresolved":false,"context_lines":[{"line_number":30,"context_line":"dynamic \u003d [\"version\", \"dependencies\"]"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"[project.optional-dependencies]"},{"line_number":33,"context_line":"bcrypt \u003d ["},{"line_number":34,"context_line":"    \"bcrypt\u003e\u003d3.1.3\","},{"line_number":35,"context_line":"]"},{"line_number":36,"context_line":"tracing-http \u003d ["}],"source_content_type":"text/x-toml","patch_set":1,"id":"01d72249_d583ffff","line":33,"in_reply_to":"4249c328_8559eaf6","updated":"2026-10-05 16:44:00.000000000","message":"Done, renamed to basic-auth in the extra, tox.ini, the install hint and the release note.","commit_id":"cb70df73c6cb5924f8c373a0a298fc32bc62913d"}]}
