lgoldstein commented on pull request #132:
URL: https://github.com/apache/mina-sshd/pull/132#issuecomment-644251850


   >> Yes, I think so. I'm a little iffy on the BaseGCMCipher name for the 
class as I'm imagining a different use of AES/GCM along the lines of how 
OpenSSH implements ChaCha20-Poly1305 which would behave differently (the 
additional authenticated data would be encrypted but not authenticated using 
another AES key using AES/CTR like how in the packet length is encrypted).
   
   We'll cross that bridge when we come to it - refactoring is not a dirty word 
in this project


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org

Reply via email to