Skip to content

Commit 63d2ac2

Browse files
committed
PR feedback
1 parent 36086d1 commit 63d2ac2

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

changes/2020-07-13_detect-base64-encoded-messages/change.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ This serves as a reference of all features that this change affects.
1010
| Feature |
1111
| ------------------------------------------------------------------------------------------------------------------------------------------- |
1212
| [Decrypt](https://github.com/awslabs/aws-encryption-sdk-specification/blob/623992d999db0b309d8a8adbd664f0d72feee813/client-apis/decrypt.md) |
13-
| |
1413

1514
## Affected Specifications
1615

@@ -52,10 +51,11 @@ the first two bytes of the Base64 encoding of a valid message are also simple to
5251

5352
## Drawbacks
5453

55-
This feature technically means the version value is capped to 40 or less,
56-
to ensure that the Base64-encoded values are not ambiguous.
54+
This feature technically means the version value
55+
cannot be incremented beyond the hex value `40`,
56+
since `41` would then conflict with the Base64-encoded value for `1`.
5757
This does not seem worth worrying about
58-
given how rarely we intended to change the version.
58+
given how rarely we intended to change the version, however.
5959

6060
## Security Implications
6161

client-apis/decrypt.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111

1212
- 0.2.0
1313

14-
- [Detect Base64-encoded Messages](../changes/2020-05-13_remove-keyring-trace/change.md)
14+
- [Detect Base64-encoded Messages](../changes/2020-07-13_detect-base64-encoded-messages/change.md)
1515

1616
- 0.1.0-preview
1717

0 commit comments

Comments
 (0)