lyft / lyft/python-blessclient
Dont add to ssh-agent unless specified
まだ誰も着手していません。
- 主要言語
- Python
- スター
- 117
- フォーク
- 31
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
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.
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
blessclient/client.py の 594-597 行目から開始し、クライアントオプションが SSH-agent の処理にどのように渡されるかを追跡します。既存の動作をデフォルトで維持しつつ、Identity ファイルの追加を無効にするパラメータを追加します。wrapper のユースケースで他の認証情報に影響を与えずに追加しないことを選択できれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- python
- 領域
- authentication, security
- issue の種類
- 機能追加
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100