lyft / lyft/python-blessclient
Dont add to ssh-agent unless specified
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 117
- Forks
- 31
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
I would suggest implementing a parameter that allows stopping bless from adding the Identity file to ssh-agent.
Namely this lines: https://github.com/lyft/python-blessclient/blob/master/blessclient/client.py#L594-L597
Given a config like:
Match exec "env | grep -q BLESS_COMPLETE || /Users/stype/blessclient/blessclient.run --gui --host '%h'"
IdentityFile ~/.ssh/blessid
this is not required and could interfere with other credentials.
I think this is required when doing a wrapper around the ssh command but in that case its easy to toggle on in the script.
If this is accepted I can create a PR to cleanup/implement this.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie in blessclient/client.py in den Zeilen 594-597 und verfolgen Sie, wie die Client-Optionen an die Behandlung des SSH-agent weitergegeben werden. Fügen Sie einen Parameter hinzu, der das Hinzufügen der Identity-Datei deaktiviert, wobei das bestehende Verhalten standardmäßig erhalten bleibt; erfüllt ist die Anforderung, wenn der Wrapper-Anwendungsfall darauf verzichten kann, ohne andere Anmeldedaten zu beeinflussen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- authentication, security
- Issue-Typ
- Feature
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100