[CAS extension] /logout should support service parameter
Description
Hi,
Currently, /logout uses the url parameter, for the return URL (defined in GeoServerCasConstants.LOGOUT_URL_PARAM). According to recent CAS documentation, it should be service, rather than url, at least on most recent versions of CAS. I suppose at some point url was the proper parameter, so the best fix would probably be to allow to configure it in the UI ?
BTW, there is a hardcoded URL here in the code of the extension. I’m pretty much certain it should not be there (and I don’t see what the returnURL param is there for)
Hi,
Currently, /logout uses the
urlparameter, for the return URL (defined in GeoServerCasConstants.LOGOUT_URL_PARAM). According to recent CAS documentation, it should beservice, rather thanurl, at least on most recent versions of CAS. I suppose at some pointurlwas the proper parameter, so the best fix would probably be to allow to configure it in the UI ?BTW, there is a hardcoded URL here in the code of the extension. I’m pretty much certain it should not be there (and I don’t see what the returnURL param is there for)
Thanks