Skip to content
This repository has been archived by the owner on Feb 23, 2024. It is now read-only.

Clarify cli login is real password, not an access token, in CLI setup tutorial #161

Closed
jpwarren opened this issue Apr 29, 2017 · 6 comments
Assignees
Labels

Comments

@jpwarren
Copy link
Contributor

I have an access token set up for providing access to my GitHub to external apps and for code push.

When doing rug login in the Tutorial, I attempted to use my GitHub access token instead of the actual password. The command silently fails, and the exit code is 0, so it looks like it succeeded, but it hasn't.

Need to clarify in the documentation to use your actual password for GitHub, not any configured access tokens.

@russmiles
Copy link
Contributor

Possibly this might be a refinement/enhancement issue for the CLI as well to perhaps do some sort of test execution when the credentials have been provided so some feedback can be given in the tool as well @cdupuis ?

@cdupuis
Copy link
Contributor

cdupuis commented Apr 30, 2017

@jpwarren how does the cli silently fail? I agree with @russmiles this being a bug in the CLI.

@jpwarren
Copy link
Contributor Author

The command simply exits, with error code 0.

If you get your credential wrong, you get an error: Provided credentials are invalid. Please try again with correct credentials. but if you use a correct auth token, you don't get this error.

It doesn't prompt for multi-factor auth (which was a clue for me, because I have MFA turned on), it just exits.

@cdupuis
Copy link
Contributor

cdupuis commented Apr 30, 2017

When it returns, did it write a github token into your ~/.atomist/cli.yml?

@jpwarren
Copy link
Contributor Author

I don't think so.

I successfully authenticated using my actual password, so I don't have the same initial conditions any more, but if I re-run the login command to test now, I get the same behaviour, and I don't see any updates to ~/.atomist/cli.yml and the file modify timestamp is the same.

@ddgenome ddgenome added the bug label May 1, 2017
ddgenome pushed a commit that referenced this issue May 1, 2017
Add admonition that you should use your GitHub password when running
`rug login`.

Addresses part of #161.
@ddgenome
Copy link
Contributor

ddgenome commented May 1, 2017

I've added text in the docs that make this more clear. I'll open an issue in rug-cli about the erroneous successful exit status and then close this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants