autoCAT6 commented on PR #54884: URL: https://github.com/apache/arrow/pull/54884#issuecomment-6354344510
Thanks, updated. I shortened the description and simplified the algorithm copy. Signing and both verification paths now explicitly use GCM. I checked the old code too: `metadata=true` already selects GCM, and `SignedFooterEncrypt` rejects CTR. The new footer tag test also passes on the previous head. So the behavior fix is the metadata value; the GCM calls just make the signing rule clear. C++ tests and the Java 1.18.1 checks pass. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
