nextcloud/core
Lukas Reschke 245080e647
Properly allow \OCP\Authentication\IApacheBackend to specify logout URL
Any `\OCP\Authentication\IApacheBackend` previously had to implement `getLogoutAttribute` which returns a string.
This string is directly injected into the logout `<a>` tag, so returning something like `href="foo"` would result
in `<a href="foo">`.

This is rather error prone and also in Nextcloud 12 broken as the logout entry has been moved with
054e161eb5 inside the navigation manager where one cannot simply inject attributes.

Thus this feature is broken in Nextcloud 12 which effectively leads to the bug described at nextcloud/user_saml#112,
people cannot logout anymore when using SAML using SLO. Basically in case of SAML you have a SLO url which redirects
you to the IdP and properly logs you out there as well.

Instead of monkey patching the Navigation manager I decided to instead change `\OCP\Authentication\IApacheBackend` to
use `\OCP\Authentication\IApacheBackend::getLogoutUrl` instead where it can return a string with the appropriate logout
URL. Since this functionality is only prominently used in the SAML plugin. Any custom app would need a small change but
I'm not aware of any and there's simply no way to fix this properly otherwise.

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
2017-08-30 14:42:46 +02:00
..
ajax Proper logging for appstore updates 2017-05-11 13:35:17 -05:00
Command Allow to specify the table space on occ 2017-08-03 16:39:33 +02:00
Controller Properly allow \OCP\Authentication\IApacheBackend to specify logout URL 2017-08-30 14:42:46 +02:00
css Merge pull request #6108 from nextcloud/backport-6080-12 2017-08-24 02:58:03 +02:00
doc changed to nextcloud 2016-06-22 08:20:23 +02:00
fonts move font back from Light to Regular for body text on normal-res screens, fix #18954 2015-11-10 16:12:42 +01:00
img Use PNG version of the icons for shipped activities 2017-07-07 11:16:44 +02:00
js Merge pull request #6127 from nextcloud/12-6126 2017-08-17 09:16:28 +02:00
l10n [tx-robot] updated from transifex 2017-08-30 00:08:36 +00:00
Middleware Check whether the $_SERVER['REQUEST_*'] vars exist before using them 2017-05-15 14:33:27 +02:00
search Remove single quotes around search query like in user search 2017-03-15 12:53:44 +01:00
skeleton ownCloud -> Nextcloud 2016-07-28 15:47:04 +02:00
templates Properly allow \OCP\Authentication\IApacheBackend to specify logout URL 2017-08-30 14:42:46 +02:00
vendor Build merged JS 2017-08-17 12:36:32 +02:00
Application.php Automatic creation of Identity manager 2017-05-10 09:45:11 +02:00
register_command.php Adding tests for 4 byte unicode characters 2017-03-21 16:42:12 -06:00
routes.php add contactsmenu popover 2017-04-26 09:26:53 +02:00
shipped.json Add missing version requirement to OAuth2 app 2017-05-22 09:01:34 +02:00
strings.php Update with robin 2016-07-21 18:13:58 +02:00