? Success

User tests: Successful: Unsuccessful:

avatar sarciszewski
sarciszewski
28 Jul 2014

AES is MCRYPT_RIJNDAEL_128, but supports keys of 16, 24, or 32 bytes.

Closes #3827

avatar sarciszewski sarciszewski - open - 28 Jul 2014
avatar Bakual
Bakual - comment - 28 Jul 2014

Please add some testing instructions and informations what you try to achieve here. Thanks.

avatar sarciszewski
sarciszewski - comment - 28 Jul 2014

Testing instructions:

  1. Not using Rijndael256 thinking it's AES256.
  2. Using AES with a 256-bit key would be AES256.

Done.

avatar Bakual
Bakual - comment - 28 Jul 2014

I honestly have no clue what you're talking about. I know it's about some encryption but where is it used and how could someone who doesn't know that stuff test it? You need to provide some testing instructions for non-developers to get some testing done.
And also some explanations what you try to do. Not everyone knows all the encryption algorythms :smile:

Can you provide that?

avatar sarciszewski
sarciszewski - comment - 28 Jul 2014

http://stackoverflow.com/questions/4128497/picking-encryption-cipher-for-mcrypt/19737721#19737721

As far as testing goes, I don't have any test vectors off-hand and I'm kind of in a hurry. Maybe ask @ircmaxell?

avatar ircmaxell
ircmaxell - comment - 28 Jul 2014

Massive -1.

Is there really a need for everyone to invent their own crypto and re-implement it? You can't use an existing implementation like Zend\Crypt???

avatar sarciszewski
sarciszewski - comment - 28 Jul 2014
avatar brianteeman brianteeman - change - 21 Aug 2014
Status New Pending
avatar nicksavov nicksavov - change - 21 Aug 2014
Labels Removed: ?
avatar sarciszewski sarciszewski - close - 7 Oct 2014
avatar sarciszewski sarciszewski - close - 7 Oct 2014
avatar sarciszewski sarciszewski - change - 7 Oct 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-10-07 14:28:10
avatar sarciszewski sarciszewski - head_ref_deleted - 7 Oct 2014

Add a Comment

Login with GitHub to post a comment