Skip to content

Commit e75c12e

Browse files
Apply suggestions from code review
Co-authored-by: Lucas McDonald <[email protected]>
1 parent 522b0f9 commit e75c12e

File tree

2 files changed

+0
-6
lines changed

2 files changed

+0
-6
lines changed

DynamoDbEncryption/runtimes/python/src/aws_dbesdk_dynamodb/internaldafny/extern/InternalLegacyOverride.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -72,10 +72,6 @@ def Build(config: DynamoDbItemEncryptorConfig_DynamoDbItemEncryptorConfig):
7272
legacy_instance = InternalLegacyOverride()
7373
legacy_instance.encryptor = legacy_override.encryptor
7474
legacy_instance.policy = legacy_override.policy
75-
# # Access the value property, not calling it as a function
76-
# legacy_instance.encryption_context = maybe_encryption_context.value
77-
# # Access the value property, not calling it as a function
78-
# legacy_instance.attribute_actions = maybe_actions.value
7975
legacy_instance.crypto_config = CryptoConfig(
8076
materials_provider=legacy_override.encryptor._materials_provider,
8177
encryption_context=maybe_encryption_context.value,

Examples/runtimes/python/Migration/src/ddbec_to_awsdbe/awsdbe/migration_step_1.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,6 @@
2424
"""
2525
from aws_dbesdk_dynamodb.structures.dynamodb import LegacyPolicy
2626

27-
# Import from new AWS Database Encryption SDK
28-
# Import from legacy DynamoDB Encryption Client
2927
from .common import setup_awsdbe_client_with_legacy_override
3028

3129

0 commit comments

Comments
 (0)