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

Introduce doctl auth init #111

Merged
merged 3 commits into from
Jul 25, 2016
Merged

Introduce doctl auth init #111

merged 3 commits into from
Jul 25, 2016

Conversation

bryanl
Copy link
Contributor

@bryanl bryanl commented Jul 23, 2016

Introduces dolt auth init command which replaces dolt auth login.

Instead of trying to fetch the token from the API, the user will have to manually enter it instead.

Also moves the default configuration location to $XDG_CONFIG_HOME/doctl/doctlcfg. On Windows, the default location is %APPDATA%/doctl/config and on other platform without the environment variable set, it will look in $HOME/.config/doctl.

If the legacy location is detected, a warning will be displayed.

Fixes #59
Fixes #94

@bryanl bryanl changed the title Introduct doctl auth init Introducte doctl auth init Jul 23, 2016
@bryanl bryanl changed the title Introducte doctl auth init Introduce doctl auth init Jul 23, 2016

func (dsa *doitServerAuth) createAuthURL(ac *doitserver.AuthCredentials, kps ...keyPair) (string, error) {
authURL, err := url.Parse(dsa.url)
// RunAuthInit initializes the doctl config. Configuring is stored in $XDG_CONFIG_HOME/doctl. On Unix, if
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/Configuring/Configuration/

@bryanl bryanl force-pushed the auth-init branch 6 times, most recently from 88f9fe7 to 458f2ac Compare July 25, 2016 02:59
@bryanl
Copy link
Contributor Author

bryanl commented Jul 25, 2016

@aybabtme: ^^

var cfgFileWriter = defaultConfigFileWriter

// ErrNoAccesToken is an error for when there is no access token.
var ErrNoAccesToken = errors.New("no access token has been configured")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/ErrNoAccesToken/ErrNoAccessToken/

@aybabtme
Copy link
Contributor

:shipit: at will

@bryanl bryanl merged commit e17a1be into master Jul 25, 2016
@bryanl bryanl deleted the auth-init branch July 25, 2016 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants