search

awslabs / aws-encryption-sdk-specification

AWS Encryption SDK Specification
Other
30 stars 27 forks source link

docs: specify how caching CMM should handle max plaintext length in Get Encryption Materials #159

Open alex-chew opened 4 years ago

alex-chew commented 4 years ago

Issue #, if available: #80 and #81

Description of changes:

The Get Encryption Materials operation accepts an optional max plaintext length parameter, but the specification does not state how the caching CMM should behave when the caller does not provide the parameter value. This change specifies that the caching CMM should bypass the cache in this case.

Also, when the caching CMM is performing a Get Encryption Materials operation for which no materials are cached, it MUST call its underlying CMM's Get Encryption Materials operation. The specification does not state what value of max plaintext length (if any) the caching CMM should pass to its underlying CMM. This change specifies that the caching CMM should pass its byte limit value as the max plaintext length parameter of the call to the underlying CMM.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

acioc commented 4 years ago

Quick follow up. Make sure not-grep and prettier both work. I suspect they're pending because of the force-push.