Skip to content

feat: support https push #353

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

Merged
merged 5 commits into from
Oct 25, 2020

Conversation

alocquet
Copy link
Contributor

Fix #344

Finally, I use git2 credential help and I retrieve username/password from :

  • git credential manager
  • url
  • git config

I show username textbox only if I can't retrieve it. I clean credentials after push to avoid keeping it in memory.

@alocquet alocquet changed the title feat: support https push WIP feat: support https push Oct 19, 2020
@alocquet alocquet force-pushed the feature/support-https-push branch from ab827cf to 6c64db2 Compare October 19, 2020 16:41
Copy link
Collaborator

@extrawurst extrawurst left a comment

Choose a reason for hiding this comment

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

Hey @alocquet thanks for working on this❤️! I have made some comments

@alocquet alocquet force-pushed the feature/support-https-push branch from 53541f3 to 2db713b Compare October 20, 2020 16:50
@alocquet alocquet force-pushed the feature/support-https-push branch from 9a9f004 to ec876e1 Compare October 20, 2020 21:04
@alocquet alocquet changed the title WIP feat: support https push feat: support https push Oct 20, 2020
@extrawurst
Copy link
Collaborator

@alocquet sorry it takes me so long to get this merged. I hope to have time tomorrow to give it a spin and then merge 🙈

@alocquet
Copy link
Contributor Author

Don't worry :)

@extrawurst extrawurst merged commit 9439114 into gitui-org:master Oct 25, 2020
@extrawurst
Copy link
Collaborator

Awesome! Works for me. Just gave it a spin! Thanks for working on this❤️!
Maybe its a little more ergonomic to not use the huge default TextInputComponent Size but that's a battle for another day. also using "".to_string() I suggest using String::new() in the future.

@extrawurst extrawurst mentioned this pull request Oct 25, 2020
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.

support https push via credentials dialog
2 participants