gary-kim/riotchat

Feature: Option to redirect back to iframed version after SSO/... login

jonathanmmm opened this issue · 1 comments

Hi,

I saw the following pull request
#235
right now this makes it possible to login via e.g. OpenID Connect, but the thing is the users are after login still on the non iframed version. This could make users unaware that they are still logged into nextcloud and even when logging out from Element they are still logged into nextcloud, as synapse does not right now support to send the log out command to an OpenID Connect server.

If users are logging out via nextcloud their session is locally deleted.

I tried that with redirects via nginx, but it seems the # makes it so it is locally (can't see server accesses via this path) and if I say /apps/riotchat/riot/ redirect to /apps/riotchat/., like this:

location ~ /apps/riotchat/riot/ {
        return  302  /apps/riotchat/$1$is_args$args;
    }

it does not work either (getting double nextcloud bar and blank white page below).

I also tried and removed /riot from main.js and main.js.map, didn't work.

My proposal would be an option:

  • To make it possible via a setting, so that when the user reaches /apps/riotchat/riot/#/home it redirects back to /apps/riotchat/#/home, that way the iframed version can be used and the user sees the nextcloud bar at the top and can log out that way.
  • This could be done via maybe a config (config.php) or via the Web UI

I changed in js/main.js and js/main.js.map the following string /apps/riotchat/riot/#/login to /apps/riotchat/#/login (before I removed all /riot parts.

Now it works, I can log in via OpenID Connect in an iFrame.

Still propose to make this possible via UI/config parameter to not have to change the source code (possibly after every update)