googleapis / googleapis/google-cloud-cpp
Refactor storage credentials
- 主要言語
- C++
- スター
- 659
- フォーク
- 462
- 平均マージ
- 1日 2時間
- マージ済み PR(30日)
- 89
説明
Storage introduced credentials in the public namespace `google::cloud::storage::oauth2`. These credential classes are now deprecated in favor of GUAC (`google::cloud::Credentials`):
https://github.com/googleapis/google-cloud-cpp/blob/80a518a357c93eec7ddc49666efcd7984135dafb/google/cloud/storage/oauth2/credentials.h#L45
However, the storage internals are implemented in terms of `google::cloud::storage::oauth2::Credentials`.
https://github.com/googleapis/google-cloud-cpp/blob/80a518a357c93eec7ddc49666efcd7984135dafb/google/cloud/storage/client.cc#L50
We map GUAC -> storage credentials.
https://github.com/googleapis/google-cloud-cpp/blob/80a518a357c93eec7ddc49666efcd7984135dafb/google/cloud/storage/client_options.cc#L296-L300
This is backwards. We need to map storage credentials to GUAC and use GUAC in the implementation, if we ever want to remove the deprecated credentials in `google::cloud::storage::oauth2`.
コントリビューションガイド
調査の方向性
google/cloud/storage/oauth2/credentials.h、google/cloud/storage/client.cc、google/cloud/storage/client_options.cc から始め、特に記載されている認証情報のマッピングを確認してください。Storage の認証情報と google::cloud::Credentials が Storage の内部処理をどのように流れるかを追跡してください。実装が GUAC の認証情報を使用し、マッピングの方向が Storage の認証情報から GUAC になっており、非推奨の公開認証情報クラスに依存していなければ完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- cpp, google-cloud
- 領域
- authentication, backend-api-design, cloud
- issue の種類
- リファクタリング
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 30/100