Skip to content
This repository has been archived by the owner on Apr 3, 2019. It is now read-only.

Push via HTTP with credentials on Windows #61

Closed
kylekatarnls opened this issue Apr 29, 2015 · 10 comments
Closed

Push via HTTP with credentials on Windows #61

kylekatarnls opened this issue Apr 29, 2015 · 10 comments

Comments

@kylekatarnls
Copy link

Hi,

I try to push my repo with the "Push" button (enabled) but nothing happen. It is a HTTP remote and it need credentials. The following command appears but nothing else:
git -c push.default=simple push --porcelain
If I type the same in command-line, it ask me my credentials and it works.

I'm on Windows.

@MarcelMue
Copy link
Collaborator

Hi @kylekatarnls ,
This seems to be the same issue as #7 .
So far our only guess is that the environment inside our shelljs shell used by git-promise doesn't get propagated properly.
I currently don't have the time to look into this issue too much as it would (most likely) need significant changes to our current setup to fix this problem. It is on our roadmap however.

Contributions or Pull requests are welcome though :D

@kylekatarnls
Copy link
Author

I got the same conclusion and was trying to propose a pull request to shelljs to handle the command read-line. Then a pull request will be needed at git-promise to handle credentials asking, then a form to display it wil be able to be done in atom-git-control. Sound like it wil take some time but I will try to do it, this feature would be very nice to me.

@MarcelMue
Copy link
Collaborator

I am not trying to be a downer but a pull request getting merged into shelljs does not seem very likely to me unfortunately. The repo has 23 open pull requests right now and in all of 2015 exactly one outside pull request has been merged.

At this point I think it would be a better idea to try and look for other options from shelljs.Maybe Atoms Buffered Process could do the job, but I am honestly not experienced enough with Node to tell :(

@kylekatarnls
Copy link
Author

https://github.com/jprichardson/node-suppose handle it but is no more up to date.

@bnb
Copy link

bnb commented Jun 22, 2015

Also having this problem. Would really like to see a fix for this.

@hellerbarde
Copy link
Contributor

@vnabet
Copy link

vnabet commented Oct 19, 2015

For windows git credentials you can use Git Crendetial Manager for Windows

Make, for instance, a pull in a command line to register your credentials. Then Push command will work fine with atom git-control

@kylekatarnls
Copy link
Author

Yes but it is not a way as secured as asking for each push the password.

@hellerbarde
Copy link
Contributor

@kylekatarnls What do you propose as an alternative? Entering passwords all the time is not a good solution either.

@kylekatarnls
Copy link
Author

Why not, if you have a password or a key with a passphrase, the most secure way to use it is to enter it each time, or each session. In fact, the most agnostic way to deal with it would be to ask the user with a prompt box for example for any input from the console. It is possible to use wincreds storage (for example with a checkbox). But IMHO, atom-git-control should let the choice to the user as git do.

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

No branches or pull requests

5 participants