aws / aws/aws-dynamodb-encryption-java
Don't print AttributeValues in EncryptionContext.toString()
- 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
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