googleapis / googleapis/google-cloud-cpp

Refactor storage credentials

Ouverte
#14,787 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
api: storage type: cleanup
Langage dominant
C++
Étoiles
659
Forks
462
Merge moyen
1 j 2 h
PR mergées (30 j)
89

Description

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`.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par google/cloud/storage/oauth2/credentials.h, google/cloud/storage/client.cc et google/cloud/storage/client_options.cc, en particulier par le mappage des identifiants cité. Suivez la manière dont les identifiants Storage et google::cloud::Credentials circulent dans les composants internes de Storage. Le travail est terminé lorsque l’implémentation utilise les identifiants GUAC et que le mappage va des identifiants Storage vers GUAC, sans s’appuyer sur les classes publiques d’identifiants obsolètes.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp, google-cloud
Domaine
authentication, backend-api-design, cloud
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
30/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.