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

RFE: PyGit2 backend should also support SSH Agent Authentication #40008

Open
The-Loeki opened this issue Mar 14, 2017 · 7 comments
Open

RFE: PyGit2 backend should also support SSH Agent Authentication #40008

The-Loeki opened this issue Mar 14, 2017 · 7 comments
Labels
Core relates to code central or existential to Salt Feature new functionality including changes to functionality and code refactors, etc. File Servers
Milestone

Comments

@The-Loeki
Copy link
Contributor

Description of Issue/Question

In Salt-SSH rosters you can already use agent-forwarding to use a key from the SSH Agent.
Using GitPython as backend for the gitfs SSH Agent auth (accidentally) also works

PyGit2 has support for this as well, but the code needs some minor adjustments to make it work.

@terminalmage
Copy link
Contributor

Do you have any examples of this? There's no mention of agent forwarding in the documentation for the Credentials objects.

@The-Loeki
Copy link
Contributor Author

libgit2/pygit2#424

@Ch3LL Ch3LL added the Pending-Discussion The issue or pull request needs more discussion before it can be closed or merged label Mar 14, 2017
@Ch3LL Ch3LL added this to the Blocked milestone Mar 14, 2017
@terminalmage
Copy link
Contributor

terminalmage commented Mar 15, 2017

Oh, cool! Odd that this has never made it into their documentation. I'll play around with this and try to get it into Salt. Thanks for the link!

I can't promise this will make the next feature release because Python 3 support is a higher priority right now, but I'll try to make it happen.

@terminalmage terminalmage added Core relates to code central or existential to Salt Feature new functionality including changes to functionality and code refactors, etc. File Servers ZRELEASED - 2018.3.0 TEAM Core and removed Pending-Discussion The issue or pull request needs more discussion before it can be closed or merged labels Mar 15, 2017
@terminalmage terminalmage self-assigned this Mar 15, 2017
@terminalmage terminalmage modified the milestones: Approved, Blocked Mar 15, 2017
@The-Loeki
Copy link
Contributor Author

That's OK :) there's obvious and good workarounds for it so no high prio.

@stale
Copy link

stale bot commented Jan 10, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

@stale stale bot added the stale label Jan 10, 2020
@The-Loeki
Copy link
Contributor Author

not stale

@stale
Copy link

stale bot commented Jan 13, 2020

Thank you for updating this issue. It is no longer marked as stale.

@stale stale bot removed the stale label Jan 13, 2020
@terminalmage terminalmage removed their assignment Jan 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core relates to code central or existential to Salt Feature new functionality including changes to functionality and code refactors, etc. File Servers
Projects
None yet
Development

No branches or pull requests

5 participants