* Add new field to the documentation
Signed-off-by: Thibault Jamet <tjamet@users.noreply.github.com>
Signed-off-by: Clif Houck <me@clifhouck.com>
* Fix codeql issue
codeql restricts logging fields from user input. Remove log to avoid complex escapes
Signed-off-by: Thibault Jamet <tjamet@users.noreply.github.com>
Signed-off-by: Clif Houck <me@clifhouck.com>
* Add a test to ensure validity of the documented cm
Currently, the documented configmap is invalid and raises error at the
time of applying it.
Ensure the configmap is valid with a unit test
Signed-off-by: Thibault Jamet <tjamet@users.noreply.github.com>
Signed-off-by: Clif Houck <me@clifhouck.com>
* rename additionalURLs for URLs
Signed-off-by: Thibault Jamet <tjamet@users.noreply.github.com>
Signed-off-by: Clif Houck <me@clifhouck.com>
* Allow responses redirected from SSO to alternate URLs
Signed-off-by: Thibault Jamet <tjamet@users.noreply.github.com>
Signed-off-by: Clif Houck <me@clifhouck.com>
* Consider dex enabled when there are additional URLs configured
Signed-off-by: Thibault Jamet <tjamet@users.noreply.github.com>
Signed-off-by: Clif Houck <me@clifhouck.com>
* Parse new URLs config from argocd-cm
Signed-off-by: Thibault Jamet <tjamet@users.noreply.github.com>
Signed-off-by: Clif Houck <me@clifhouck.com>
* Detect additional SSO URLs from requests
With HTTP servers, the `url` field of `http.Requests` does not contain
the host nor the protocol.
To work around this, consider `request.Host` and
`request.URL.RequestURI()`
Signed-off-by: Thibault Jamet <tjamet@users.noreply.github.com>
Signed-off-by: Clif Houck <me@clifhouck.com>
* Handle logout properly
Currently, while login is correctly handled, when logging out from
an alternate URL, we are redirected to the main URL.
Fix this by applying the same principles as for the login part
Signed-off-by: Thibault Jamet <tjamet@users.noreply.github.com>
Signed-off-by: Clif Houck <me@clifhouck.com>
* Fix ClientApp.oath2Config
Signed-off-by: Clif Houck <me@clifhouck.com>
* Fix test compile time error
Signed-off-by: Clif Houck <me@clifhouck.com>
* Fix TestDocumentedArgoCDConfigMapIsValid to parse in the same way as other spots in the codebase
Signed-off-by: Clif Houck <me@clifhouck.com>
* Fix warning messages for external URLs when attemping to load them from config map
Signed-off-by: Clif Houck <me@clifhouck.com>
* Revert change to IsDexConfigured since Dex does not use URLs setting
Signed-off-by: Clif Houck <me@clifhouck.com>
* Remove unnecessary receiver nil check in favor of fixing test
Signed-off-by: Clif Houck <me@clifhouck.com>
* Fix typo
Signed-off-by: Clif Houck <me@clifhouck.com>
* Add a unit test for RedirectURLForRequest
Signed-off-by: Clif Houck <me@clifhouck.com>
* Rename settings.URLs to AdditionalURLs
Signed-off-by: Clif Houck <me@clifhouck.com>
* Fix use of URLs in TestClientApp_HandleLogin
Signed-off-by: Clif Houck <me@clifhouck.com>
* Renamed urls to additionalUrls
Signed-off-by: Clif Houck <me@clifhouck.com>
* Integrate settings.AdditionalURLs with dex config and test settings.RedirectAdditionalURLs
Signed-off-by: Clif Houck <me@clifhouck.com>
* Reload ArgoCDServer when settings.AdditionalURLs changes
Signed-off-by: Clif Houck <me@clifhouck.com>
* Add note about additionalUrls to user-managament docs
Signed-off-by: Clif Houck <me@clifhouck.com>
* Add G-Research
Signed-off-by: Clif Houck <me@clifhouck.com>
* Change G-Research URL to point to open-source page
Signed-off-by: Clif Houck <me@clifhouck.com>
* Update docs/operator-manual/argocd-cm.yaml
Correct misspelling of additionalUrls
Co-authored-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>
Signed-off-by: Clif Houck <me@clifhouck.com>
* Update util/settings/settings.go
Fix incorrect key for additionalUrls
Co-authored-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>
Signed-off-by: Clif Houck <me@clifhouck.com>
* Update util/settings/settings.go
Fix key name
Co-authored-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>
Signed-off-by: Clif Houck <me@clifhouck.com>
* fix additional urls config key in test config
Signed-off-by: Clif Houck <me@clifhouck.com>
---------
Signed-off-by: Thibault Jamet <tjamet@users.noreply.github.com>
Signed-off-by: Clif Houck <me@clifhouck.com>
Co-authored-by: Thibault Jamet <tjamet@users.noreply.github.com>
Co-authored-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>
* feat: regenerate active users token if it is expiring soon
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
* Comment how 'renew-token' header is used
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
