feat(5388): Allow multiple external URLs for SSO access#18927

Merged

ishitasequeira merged 29 commits intoargoproj:masterfrom

ClifHouck:clif/tjamet/multiple-url-external-sso

Jul 30, 2024

Commits on Jul 29, 2024

Add new field to the documentation

tjamet
authored and
ClifHouck
committed
Fix codeql issue

tjamet
authored and
ClifHouck
committed
Add a test to ensure validity of the documented cm

tjamet
authored and
ClifHouck
committed
rename additionalURLs for URLs

tjamet
authored and
ClifHouck
committed
Allow responses redirected from SSO to alternate URLs

tjamet
authored and
ClifHouck
committed
Consider dex enabled when there are additional URLs configured

tjamet
authored and
ClifHouck
committed
Parse new URLs config from argocd-cm

tjamet
authored and
ClifHouck
committed
Detect additional SSO URLs from requests

tjamet
authored and
ClifHouck
committed
Handle logout properly

tjamet
authored and
ClifHouck
committed
Fix ClientApp.oath2Config
ClifHouck
committed
Fix test compile time error
ClifHouck
committed
Fix TestDocumentedArgoCDConfigMapIsValid to parse in the same way as other spots in the codebase
ClifHouck
committed
Fix warning messages for external URLs when attemping to load them from config map
ClifHouck
committed
Revert change to IsDexConfigured since Dex does not use URLs setting
ClifHouck
committed
Remove unnecessary receiver nil check in favor of fixing test
ClifHouck
committed
Fix typo
ClifHouck
committed
Add a unit test for RedirectURLForRequest
ClifHouck
committed
Rename settings.URLs to AdditionalURLs
ClifHouck
committed
Fix use of URLs in TestClientApp_HandleLogin
ClifHouck
committed
Renamed urls to additionalUrls
ClifHouck
committed
Integrate settings.AdditionalURLs with dex config and test settings.RedirectAdditionalURLs
ClifHouck
committed
Reload ArgoCDServer when settings.AdditionalURLs changes
ClifHouck
committed
Add note about additionalUrls to user-managament docs
ClifHouck
committed
Add G-Research
ClifHouck
committed
Change G-Research URL to point to open-source page
ClifHouck
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(5388): Allow multiple external URLs for SSO access#18927

feat(5388): Allow multiple external URLs for SSO access#18927
ishitasequeira merged 29 commits intoargoproj:masterfrom
ClifHouck:clif/tjamet/multiple-url-external-sso

Commits on Jul 29, 2024

Add new field to the documentation

Fix codeql issue

Add a test to ensure validity of the documented cm

rename additionalURLs for URLs

Allow responses redirected from SSO to alternate URLs

Consider dex enabled when there are additional URLs configured

Parse new URLs config from argocd-cm

Detect additional SSO URLs from requests

Handle logout properly

Fix ClientApp.oath2Config

Fix test compile time error

Fix TestDocumentedArgoCDConfigMapIsValid to parse in the same way as other spots in the codebase

Fix warning messages for external URLs when attemping to load them from config map

Revert change to IsDexConfigured since Dex does not use URLs setting

Remove unnecessary receiver nil check in favor of fixing test

Fix typo

Add a unit test for RedirectURLForRequest

Rename settings.URLs to AdditionalURLs

Fix use of URLs in TestClientApp_HandleLogin

Renamed urls to additionalUrls

Integrate settings.AdditionalURLs with dex config and test settings.RedirectAdditionalURLs

Reload ArgoCDServer when settings.AdditionalURLs changes

Add note about additionalUrls to user-managament docs

Add G-Research

Change G-Research URL to point to open-source page

Commits on Jul 30, 2024

Update docs/operator-manual/argocd-cm.yaml

Update util/settings/settings.go

Update util/settings/settings.go

fix additional urls config key in test config