Skip to content

Conversation

@udon-or-ramen
Copy link
Contributor

概要

  • 取得したトークンを平文Jsonの代わりにzalando/go-keyringを使用して、各環境のセキュアな資格情報マネージャーに保存します
  • aurl上のProfile名を資格情報のユーザーとして保存し、パスワードにgo-keyringによってBase64に変換されたトークンデータを挿入することで簡易的な暗号化とする狙いです。

懸念

  • Go言語なので問題ないかと思いますが、Win環境が手元になく正しく動作するか確認できていません。
    • (MacOSでは問題なくキーチェーンに保存されていることを確認しています)
  • /usr/bin/security からキーチェーンにアクセスするため、aurlが要求しているとわかりにくい可能性があります。
    • スクリーンショット 2025-12-01 14 54 27
  • TouchIDを認証に使用できないため、利便性に問題があるかもしれません

Copy link
Collaborator

@seiichi1101 seiichi1101 left a comment

Choose a reason for hiding this comment

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

ありがとうございます!macとwindowsでの動作確認問題なかったです。
僕の wsl(Ubuntu環境)では keychain をうまく参照できなかったみたいなので、ちょっと工夫がいるかもです。

いずれにせよ、リポジトリを少し整えて、 今後の master では aurl-v2-alpha として進めていこうと思うので、一旦こちらをマージしたいと思います。

(こちらは余談ですが、 keyringのモジュールもできれば、 https://github.com/ByteNess/keyring の方がカバー範囲が広そうなので、そっちを使いたい気持ちもあります。 自分たちでメンテナンスをしていく覚悟が必要ですが。。。

@seiichi1101 seiichi1101 merged commit 1f2ae8f into classmethod:master Dec 8, 2025
@udon-or-ramen
Copy link
Contributor Author

レビューとマージありがとうございます!
keyringモジュールの選定については
Github CLIでの採用例があり信用できそうだなと言う理由ですので、変更に関しては適宜といった感じで...(本当は、Touch IDのための実装があるのでByteness/keyringの方が確実に嬉しいです)

@udon-or-ramen udon-or-ramen deleted the feat/use-keyring branch December 9, 2025 01:02
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.

2 participants