)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8648,"name":"Radomir Dopieralski","email":"openstack@dopieralski.pl","username":"thesheep"},"change_message_id":"29f3e972a6bbd18045dae93d56570e4cae849ee7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"a6924004_d7184b20","updated":"2026-07-30 15:12:10.000000000","message":"Do we really need 3 different settings for this? I understand the last one is legacy and already in there, so we can leave it there, but the other two seem to be redundant? It\u0027s exactly this situation of adding more and more specific settings that I wanted to avoid by doing the research into what is exactly needed up front. We don\u0027t have to rely on the original patch.","commit_id":"a09476af4db47d37c8c216c4dc1693f5757dcf1e"},{"author":{"_account_id":32432,"name":"Owen McGonagle","email":"omcgonag@redhat.com","username":"omcgonag"},"change_message_id":"d2a71acec7c848a2ac8ba01fb56a0fa70625b054","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"da2d7232_5bf1c80d","updated":"2026-07-30 17:19:22.000000000","message":"I do see having 1 and 2 could be collapsed into just supporting a redirect mapping. When you say \"research\" - are you suggesting that this method of supporting a mapping would not work, and more research is necessary. Can you list a few bullet items where more research would help?","commit_id":"a09476af4db47d37c8c216c4dc1693f5757dcf1e"},{"author":{"_account_id":32432,"name":"Owen McGonagle","email":"omcgonag@redhat.com","username":"omcgonag"},"change_message_id":"2fa8de64b8866974a46a8e5580a1850b9830dd73","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"2f125d69_161f8746","updated":"2026-08-05 15:13:19.000000000","message":"setting this to WIP until we get more info and decide best approach moving forward for single vs. multi-provider SSO logout support","commit_id":"a09476af4db47d37c8c216c4dc1693f5757dcf1e"},{"author":{"_account_id":32432,"name":"Owen McGonagle","email":"omcgonag@redhat.com","username":"omcgonag"},"change_message_id":"71449beadce78a2776da590126d8992d23998a44","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"894c5a3b_bc69826f","updated":"2026-07-29 19:42:00.000000000","message":"we will not get a +1 zuul job until https://review.opendev.org/c/openstack/horizon/+/996428 gets a +1 and merged - then this review can be merged and ready for full (testing) review.","commit_id":"a09476af4db47d37c8c216c4dc1693f5757dcf1e"},{"author":{"_account_id":36319,"name":"Freerk-Ole Zakfeld","display_name":"Freerk-Ole Zakfeld","email":"freerk@zakfeld.com","username":"freerkzakfeld","status":"I work as a cloud engineer at ScaleUp Technologies"},"change_message_id":"ad65175c82b2994909bb6dd932c3f67a88ecb4a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"2a37a221_5055e3dc","in_reply_to":"da2d7232_5bf1c80d","updated":"2026-07-31 08:52:42.000000000","message":"If my patch (WEBSSO_POST_LOGOUT_URL) does not get merged then I don\u0027t see a point adding this here, WEBSSO_LOGOUT_REDIRECT_MAPPING[auth_type] is fine. Thanks for extending it!","commit_id":"a09476af4db47d37c8c216c4dc1693f5757dcf1e"}],"openstack_auth/defaults.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ba00c8ce5128af05642456d15948b909e14ac942","unresolved":false,"context_lines":[{"line_number":158,"context_line":"# multiple authentication methods, use WEBSSO_LOGOUT_REDIRECT_MAPPING instead."},{"line_number":159,"context_line":"WEBSSO_DEFAULT_REDIRECT_LOGOUT \u003d None"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"# Map authentication methods to their logout URLs for multi-provider deployments."},{"line_number":162,"context_line":"# Enables per-provider logout redirect when multiple authentication methods"},{"line_number":163,"context_line":"# are configured. Keys must match values from WEBSSO_CHOICES or WEBSSO_IDP_MAPPING."},{"line_number":164,"context_line":"# The logout view will read the user\u0027s session \u0027auth_type\u0027 to determine which"}],"source_content_type":"text/x-python","patch_set":1,"id":"17d8ca69_f5b88b7f","line":161,"updated":"2026-07-28 16:10:28.000000000","message":"pep8: E501 line too long (81 \u003e 80 characters)","commit_id":"47dc7839ad5beda7c38089239a6b706d4f1b808a"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ba00c8ce5128af05642456d15948b909e14ac942","unresolved":false,"context_lines":[{"line_number":160,"context_line":""},{"line_number":161,"context_line":"# Map authentication methods to their logout URLs for multi-provider deployments."},{"line_number":162,"context_line":"# Enables per-provider logout redirect when multiple authentication methods"},{"line_number":163,"context_line":"# are configured. Keys must match values from WEBSSO_CHOICES or WEBSSO_IDP_MAPPING."},{"line_number":164,"context_line":"# The logout view will read the user\u0027s session \u0027auth_type\u0027 to determine which"},{"line_number":165,"context_line":"# logout URL to redirect to."},{"line_number":166,"context_line":"# Example:"}],"source_content_type":"text/x-python","patch_set":1,"id":"4d6cf8f8_31d48b59","line":163,"updated":"2026-07-28 16:10:28.000000000","message":"pep8: E501 line too long (83 \u003e 80 characters)","commit_id":"47dc7839ad5beda7c38089239a6b706d4f1b808a"}],"openstack_auth/views.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ba00c8ce5128af05642456d15948b909e14ac942","unresolved":false,"context_lines":[{"line_number":300,"context_line":"            auth_user.unset_session_user_variables(request)"},{"line_number":301,"context_line":"            return django_http.HttpResponseRedirect(logout_url)"},{"line_number":302,"context_line":"        else:"},{"line_number":303,"context_line":"            LOG.debug(\u0027Auth type \"%s\" not found in WEBSSO_LOGOUT_REDIRECT_MAPPING, \u0027"},{"line_number":304,"context_line":"                      \u0027falling back to local logout\u0027, auth_type)"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"    # Fall back to existing single-provider logout redirect"}],"source_content_type":"text/x-python","patch_set":1,"id":"6ea8eb63_f38a7033","line":303,"updated":"2026-07-28 16:10:28.000000000","message":"pep8: E501 line too long (84 \u003e 80 characters)","commit_id":"47dc7839ad5beda7c38089239a6b706d4f1b808a"}]}