PHP 7 Support

[ahmeier]

I tried to compile this for php 7, but it does not seem to work. compiles with php 7, but the scrypt.so file is not used and does not show up in phpinfo.php, even though it is in the extension directory.

[DomBlack]

I just added PHP 7 to the build matrix on Travis. It looks like one of the PHP macros has changed;
https://travis-ci.org/DomBlack/php-scrypt/jobs/92673120

I'll check it out when I get time (See #40)

[kingkorf]

See #41 also

[cbj4074]

Firstly, thank you for creating this, @DomBlack . I deeply appreciate the work and expertise that you've put into this library.

What can I do to help you prepare this for PHP 7?

And what about windows binary id like to test on windows

[redthor]

I'm just looking at running a php7 test.

If I re-apply @kingkorf's changes to php-scrypt.c in master (i.e. line 199, rather than 181) I get
warning: initialization from incompatible pointer type
when I make

Not sure if this is significant.

[cbj4074]

@redthor Does the extension compile successfully, despite the warning? Is it usable thereafter?

[redthor]

It compiles but I ran into a memory alloc issue when testing. I can give more info later. Cheers

[cbj4074]

@redthor Thanks!

This extension is the only thing keeping me from upgrading to PHP 7. My application relies on it and I lack the specialized knowledge required to massage the source into shape for PHP 7. Otherwise, I'd jump on it myself. :/

But at this point, I'm considering learning whatever I need to learn to make it work. Please let me know if I can facilitate your efforts in any way, and I'll do my best!

[redthor]

I've now changed the code to this:

diff --git a/php_scrypt.c b/php_scrypt.c
index e4c8e0e..82e884a 100644
--- a/php_scrypt.c
+++ b/php_scrypt.c
@@ -199,10 +199,10 @@ PHP_FUNCTION(scrypt)
         php_hash_bin2hex(hex, buf, keyLength);
         efree(buf);
         hex[keyLength*2] = '\0';
-        RETURN_STRINGL(hex, keyLength * 2, 0);
+        RETURN_STRINGL(hex, keyLength * 2);
     } else {
         buf[keyLength] = '\0';
-        RETURN_STRINGL((char *)buf, keyLength, 0);
+        RETURN_STRINGL((char *)buf, keyLength);
     }
 }
 /* }}} */

i.e. just removed the last 0 and it appears to be happy. More testing todo

[redthor]

@maki-chan has already done this in #45

[joshuataylor]

The following works on Ubuntu (and probably other distros as well) to on PHP7 for testing:

git clone https://github.com/DomBlack/php-scrypt.git
cd php-scrypt
wget https://patch-diff.githubusercontent.com/raw/DomBlack/php-scrypt/pull/45.patch
patch -p1 < 45.patch
phpize
./configure --enable-scrypt
make -j4
sudo make install

Then add:

extension=scrypt.so

to your conf.d files

[cbj4074]

Thank you, @maki-chan , @redthor and @joshuataylor for pointing the rest of us to a working solution.

Hopefully, @DomBlack is still alive.

Certainly not to sound ungrateful, but this 6-month setback is all the motivation I need to migrate to the native password_* family of PHP functions.

[DomBlack]

Sorry I've been really busy recently without time to look into this. There is an open PR but I've not pulled it yet because I need to maintain backwards compatibly with PHP 5 as well.

I had planned to take that PR and put predef's in it to keep backwards compability, if somebody with time and a working PHP 7 enviroment could make that change, I will pull and publish an updated version to pecl.

(I had hopped that the password functions would include scrypt, but unfortunately that never happened)