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

perforce login issues #1

Open
mattsawyer77 opened this issue Jul 11, 2013 · 4 comments · May be fixed by #25
Open

perforce login issues #1

mattsawyer77 opened this issue Jul 11, 2013 · 4 comments · May be fixed by #25

Comments

@mattsawyer77
Copy link

I attempted to use the perforce login command (which allowed my password to be displayed in clear text, minor issue there). It never seems to be able to login. I don't see any errors emitted from the login attempt, but when I try to do something subsequently, I see the following in the console:

Perforce: Perforce password (P4PASSWD) invalid or unset.
@ericmartel
Copy link
Owner

Hi Matt,

I will consider adding a warning for the password, unfortunately ST doesn't provide plugin developers with masked inputs.

I will look into your issue, I am not the person who developed the Login command, but I'll try to see tonight what I can do.

Thanks

Eric

@mattsawyer77
Copy link
Author

Thanks Eric.

@mattsawyer77
Copy link
Author

I would also suggest having the plugin look into the local ticket file first before asking for the password. That way if you're already logged in from the terminal or some other client, no need to login via Sublime Text.

For reference: http://www.perforce.com/perforce/doc.current/manuals/cmdref/login.html

@ericmartel
Copy link
Owner

Thanks for the reference!

The plugin was initially based only on p4 info (command line command) which works pretty well in most cases.

I'll look into adding the new feature, in the meantime I would suggest using p4 passwd (http://www.perforce.com/perforce/doc.current/manuals/cmdref/env.P4PASSWD.html) from the command line.

I'll keep this issue opened until I have more time to adjust the plugin

Thank you

@ymengesha ymengesha linked a pull request Aug 16, 2018 that will close this issue
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 a pull request may close this issue.

2 participants