-
Notifications
You must be signed in to change notification settings - Fork 163
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: make the cli runnable on windows #326
Conversation
Signed-off-by: Guilherme Macedo <guilherme@gmacedo.com>
Bump Go and Dockerfile versions
Signed-off-by: Guilherme Macedo <guilherme@gmacedo.com>
[v2.7] Update saml auth login endpoint to point toward dashboard endpoint
👋 @bakito Thanks for raising this PR, and apologies for the long delay in any response. If you're still willing, I'd be happy to work with you to get this PR into shape so it can be merged. My only concern being the use of If you could update this PR so that it no longer uses |
Signed-off-by: bakito <github@bakito.ch>
This should do the trick. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for the contribution
As as already mentioned in #306, the cli is not running properly in windows, as the parent dir of the config file is not evaluated correctly.
This PR fixes this and uses also
os.UserHomeDir()
to evaluate the home dir of the user instead of just expanding the HOME variable.