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

Fetching orgin from the remote branches on first load asks for the ssh key #6047

Closed
bhcoder opened this issue Jan 7, 2019 · 7 comments
Closed
Assignees
Labels
area: left-panel Issues related to the Left Panel type: bug 🐛
Milestone

Comments

@bhcoder
Copy link

bhcoder commented Jan 7, 2019

Current behaviour

Trying to fetch the origin from the branches view asks to load the ssh key
(No supported authentication methods available (server sent: public key)

It seems that the key is not loaded, while fetching first the master from the commit list will load the key correctly making the following fetch from the remote tree to work

Expected behaviour

The branch should be fetched.

Steps to reproduce

  1. close Git Extensions and Pageant
  2. open a repository which uses ssh
  3. fetch the origin from the remote branches tree on the left

Screenshots

Image

@RussKie
Copy link
Member

RussKie commented Jan 7, 2019

@vbjay has provided a number of detailed responses to the similar issues in #4916. Please read #4916 (comment) onwards and see if those resolve your issues.

@bhcoder
Copy link
Author

bhcoder commented Jan 7, 2019

The setup is the same and works if I pull/fetch from the commit list in the center:
doing that way the pageant is launched, the key is added and the fetch/pull works.
If I fetch the repo from the left tree after that it will work since the key is loaded.

However if I open the repo for the first time and try to fetch from the remote tree it will not work since the fetch command does not launch pageant and does not add the key from the repo settings asking you to manually load it

Image

@vbjay
Copy link
Contributor

vbjay commented Jan 7, 2019

That context menu should be calling the same code as the fetch button on the toolbar. We shouldn't be duplicating code and instead call a preformFetch method that gets called by the fetch menu option and if the button default is set to fetch. Same with other context menus.

@amaiorano @EbenZhang

@gerhardol gerhardol added type: bug 🐛 area: left-panel Issues related to the Left Panel labels Jan 7, 2019
@amaiorano
Copy link
Contributor

Yes, as @vbjay says, the fetch code for the Remote node in the left panel isn't the same as FormBrowse. Just tested a quick fix, and it seems to work. Will open an PR.

@RussKie
Copy link
Member

RussKie commented Jan 8, 2019

@bhcoder
Copy link
Author

bhcoder commented Jan 8, 2019

Could you please try this build and confirm the fix
https://ci.appveyor.com/api/buildjobs/ree3ecpje36yo71d/artifacts/Setup%2FGitExtensions-Portable-3.01.00.4753.zip

When opening this version I get the following error:

Could not load file or assembly 'Microsoft.VisualStudio.Composition, Version=15.6.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified.

Besides that remotes > origin > fetch works now, but remotes > origin > [branch_name] > fetch/fetch & [commmand] have the same issue

@ghost ghost assigned RussKie Jan 9, 2019
@RussKie
Copy link
Member

RussKie commented Jan 9, 2019

That issue is tracked in #5990 and is getting fixed in #6005

@ghost ghost removed the status: ready label Jan 10, 2019
@RussKie RussKie added this to the 3.1.0 milestone Jan 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: left-panel Issues related to the Left Panel type: bug 🐛
Projects
None yet
Development

No branches or pull requests

5 participants