[GH-ISSUE #156] php 7.1 issue #100

New issue

Closed

opened 2026-02-28 01:20:59 +03:00 by kerem · 5 comments

kerem commented

2026-02-28 01:20:59 +03:00

Owner

Originally created by @jostreff on GitHub (Aug 17, 2017).
Original GitHub issue: https://github.com/tuxis-ie/nsedit/issues/156

Dear All,

have anyone tryed this great web interface running on php7?

My attempt to use it under php 7.1.8 was failure.

Originally created by @jostreff on GitHub (Aug 17, 2017). Original GitHub issue: https://github.com/tuxis-ie/nsedit/issues/156 Dear All, have anyone tryed this great web interface running on php7? My attempt to use it under php 7.1.8 was failure.

kerem

2026-02-28 01:20:59 +03:00

closed this issue
added the
enhancement

help wanted
labels

kerem commented

2026-02-28 01:21:00 +03:00

Author

Owner

@tuxis-ie commented on GitHub (Oct 5, 2017):

Haven't tried that yet, no.
Feel free to debug and send in a PR

@tuxis-ie commented on GitHub (Oct 5, 2017): Haven't tried that yet, no. Feel free to debug and send in a PR

kerem commented

2026-02-28 01:21:00 +03:00

Author

Owner

@metrax commented on GitHub (Dec 23, 2017):

I'm using PHP 7.1.12 and had no problems yet.

@metrax commented on GitHub (Dec 23, 2017): I'm using PHP 7.1.12 and had no problems yet.

kerem commented

2026-02-28 01:21:00 +03:00

Author

Owner

@rainbowskydancer commented on GitHub (Feb 24, 2018):

On 7.2 works fine too, except I had to install MCrypt via PECL since Debian doesn't have it ready for 7.2 in sid yet.

@rainbowskydancer commented on GitHub (Feb 24, 2018): On 7.2 works fine too, except I had to install MCrypt via PECL since Debian doesn't have it ready for 7.2 in sid yet.

kerem commented

2026-02-28 01:21:00 +03:00

Author

Owner

@ghost commented on GitHub (Mar 9, 2018):

have it running on PHP 7.0 and no issues so far... the thing between 7.0 and 7.1 should effectively only be mcrypt as @ericcotelnu said.

Please see http://php.net/manual/de/migration71.deprecated.php for details and why PHP is dropping mcrypt (it is out of date). Seems the update should be easy enough though: it would be enough to change the enc_secret + dec_secret functions in includes/session.inc.php to something else. IMHO you'd ideally drop the dec_secret completely (no auto-login... that's a bad idea anyway) and exchange the enc_secret with just a simple hash() call...

@ghost commented on GitHub (Mar 9, 2018): have it running on PHP 7.0 and no issues so far... the thing between 7.0 and 7.1 should effectively only be mcrypt as @ericcotelnu said. Please see http://php.net/manual/de/migration71.deprecated.php for details and why PHP is dropping mcrypt (it is out of date). Seems the update should be easy enough though: it would be enough to change the enc_secret + dec_secret functions in includes/session.inc.php to something else. IMHO you'd ideally drop the dec_secret completely (no auto-login... that's a bad idea anyway) and exchange the enc_secret with just a simple hash() call...

kerem commented

2026-02-28 01:21:00 +03:00

Author

Owner

@jostreff commented on GitHub (Sep 6, 2025):

It works perfectly now I believe. I have it using using new versions of php including php 8.4.12 - everywhere it works perfectly!

@jostreff commented on GitHub (Sep 6, 2025): It works perfectly now I believe. I have it using using new versions of php including php 8.4.12 - everywhere it works perfectly!