Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(argo-workflows): Fix OAuth redirect URL autoconfig. #2369

Merged
merged 2 commits into from
Dec 7, 2023

Conversation

vladlosev
Copy link
Collaborator

Checklist:

  • I have bumped the chart version according to versioning
  • I have updated the documentation according to documentation
  • I have updated the chart changelog with all the changes that come with this pull request according to changelog.
  • Any new values are backwards compatible and/or have sensible default.
  • I have signed off all my commits as required by DCO.
  • My build is green (troubleshooting builds).

This change deletes explicit configuration of the OAuth redirect URL, which was suppressing automatic configuration of the redirect URL in the server. The default URL is incorrect for most installations anyway, which means the users cannot use the default, which is inconvenient.

Signed-off-by: Vlad Losev <vlad@losev.com>
@vladlosev vladlosev force-pushed the fix-oauth-redirect-autoconfig branch from eafc913 to 5a26eed Compare December 6, 2023 22:46
@vladlosev vladlosev changed the title Fixes OAuth redirect URL autoconfig. fix(argo-workflows): Fix OAuth redirect URL autoconfig. Dec 6, 2023
Signed-off-by: Vlad Losev <vlad@losev.com>
@vladlosev vladlosev force-pushed the fix-oauth-redirect-autoconfig branch from 7eaea47 to 8a444cc Compare December 6, 2023 22:53
@jmeridth jmeridth merged commit 690e83b into argoproj:main Dec 7, 2023
6 checks passed
@vladlosev vladlosev deleted the fix-oauth-redirect-autoconfig branch December 7, 2023 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants