aws / aws/aws-dynamodb-encryption-java

Don't print AttributeValues in EncryptionContext.toString()

Ouverte
#100 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Java
Étoiles
141
Forks
74
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

https://github.com/aws/aws-dynamodb-encryption-java/blob/master/sdk1/src/main/java/com/amazonaws/services/dynamodbv2/datamodeling/encryption/EncryptionContext.java#L220

https://github.com/aws/aws-dynamodb-encryption-java/blob/master/sdk2/src/main/java/software/amazon/cryptools/dynamodbencryptionclientsdk2/encryption/EncryptionContext.java#L182

A safe `toString()` method for `EncryptionContext` would be extremely useful for logging within material providers, and we can get there by excluding attributeValues from being printed. Since it is likely that tests rely on this today, we should consider doing this with a minor bump. (Note: We haven't published the sdk2 version yet, so it's not necessary to do a bump there.)

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Comparez les fichiers EncryptionContext.java de SDK1 et SDK2 aux emplacements toString() indiqués par les liens, puis examinez les tests existants qui couvrent leur sortie de chaîne actuelle. Excluez attributeValues de la représentation journalisée tout en conservant le reste du contexte, et prenez en compte l’incrément de version mineure de SDK1 ; aucun incrément de SDK2 n’est nécessaire puisqu’il n’a pas été publié.

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

Évaluation

Stack technique
java
Domaine
security
Type d'issue
Fonctionnalité
Difficulté
2/5
Temps estimé
Une demi-journée
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
55/100

Recevez les nouvelles issues par e-mail

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