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 Udash login flow with access_token #1436

Merged
merged 2 commits into from
Jul 31, 2023
Merged

Conversation

olblak
Copy link
Member

@olblak olblak commented Jul 31, 2023

PKCE flow doesn't work from none interactive environment such as CI
This commit allows to skip the PKCE flow if an access token is provided via parameter

Test

To test this pull request, you can run the following commands:

cp <to_package_directory>
go test

Additional Information

Tradeoff

Potential improvement

PKCE flow doesn't work from none interactive environmwent
such as CI
This commit allows to skip the PKCE flow if an access token
is provided via parameter

Signed-off-by: Olblak <me@olblak.com>
@olblak olblak added bug Something isn't working experimental Experimental feaure udash All things related to Udash labels Jul 31, 2023
Signed-off-by: Olblak <me@olblak.com>
@olblak olblak merged commit f8442f4 into updatecli:main Jul 31, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working experimental Experimental feaure udash All things related to Udash
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant