googleapis / googleapis/google-cloud-cpp

Refactor storage credentials

オープン
#14,787 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
api: storage type: cleanup
主要言語
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。