Description of changes:
Warning Backwards Incompatible change: moved gcm tag out of the constructor into a separate setter, since it was leading to a whole bunch of annoying corner cases, that was not handled consistently cross platform. And arguably having tag as a separate setter is a bit cleaner design, since we already complicate logic, by abstracting away encryptor vs decryptor
Outside of that, changes are mostly extra tests and corresponding changes to make everything consistent cross platform
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Issue #, if available:
Description of changes: Warning Backwards Incompatible change: moved gcm tag out of the constructor into a separate setter, since it was leading to a whole bunch of annoying corner cases, that was not handled consistently cross platform. And arguably having tag as a separate setter is a bit cleaner design, since we already complicate logic, by abstracting away encryptor vs decryptor
Outside of that, changes are mostly extra tests and corresponding changes to make everything consistent cross platform
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.