aws / aws/aws-dynamodb-encryption-java

Don't print AttributeValues in EncryptionContext.toString()

Offen
#100 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Java
Sterne
141
Forks
74
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Vergleiche die Dateien EncryptionContext.java von SDK1 und SDK2 an den verlinkten toString()-Stellen und prüfe anschließend die vorhandenen Tests, die ihre aktuelle String-Ausgabe abdecken. Schließe attributeValues aus der protokollierten Darstellung aus, während der übrige Kontext erhalten bleibt, und berücksichtige die Anhebung der Minor-Version von SDK1; für SDK2 ist keine Anhebung erforderlich, da es noch nicht veröffentlicht wurde.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
security
Issue-Typ
Feature
Schwierigkeit
2/5
Geschätzter Aufwand
Ein halber Tag
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.