- Issue created by @mingsong
- Status changed to Needs review
over 1 year ago 4:56am 6 April 2023 - Status changed to Postponed: needs info
over 1 year ago 4:47pm 10 August 2023 - 🇳🇱Netherlands roderik Amsterdam,NL / Budapest,HU
The title says "menu link" but the problem/motivation says "path" -- and the patch changes the routing.yml, not a menu link.
Yes, the /saml/logout path is accessible to anonymous users. That has been done explicitly in #3043704: Make user logout more robust → . If someone who is already logged out from the Drupal site gets redirected from the IdP as part of a multi-site SLO flow, then that SLO should not be interrupted (the user should get redirected back to the IdP).
The menu link provided by this module works, because it switches between "log in" and "log out" according to the user's status.
I suspect you added a 'simple' menu link with path /saml/logout through the admin UI, and want it to automatically disappear when the user is logged out. Unfortunately you cannot do this by blocking access to the /saml/logout route, because of the above.
Please close or provide more info as appropriate.
- First commit to issue fork.
- Open on Drupal.org →Core: 9.5.5 + Environment: PHP 7.4 & MySQL 5.7last update
6 months ago Waiting for branch to pass