API Request : encrypt(byte[] plainText, byte[] passwordOrKey) //return encrypted text about conceal HOT 2 CLOSED

I thought about this and I understand why this makes it simpler for your use case. However I think this API breaks the contract of Crypto and may make it more confusing to use for others. That is you pass in a keychain from which Crypto gets a key, but then the key is overriden by this API. It would also cause an explosion of methods in the case where if we have to do this for MACs as well. It would also prevents us from changing the algorithm in the future because some algorithms might supply more than 1 key as a parameter.

Hope this makes sense and represents compelling reasons not to add this API.

from conceal.

I'm closing the issue for now, feel free to reopen if you have additional comments.

from conceal.