Not compatible with php 7.2 or higher

[funkyduke]

Filelister stops working when I upgrade from php 7.1 to 7.2 or 7.3. Rolling back to 7.1 makes it work again.

[chrisandy]

My understanding is that Filelister uses mcrypt which isn't installed by default in php 7.2. Would be good to see Filelister updated as it works well otherwise.

[funkyduke]

Don't think that's the reason, because mcrypt is installed here and it's not working with php 7.2. or higher.

Also my opinion about mcrypt: There are a few important reasons why you do not want to use Mcrypt. The most important reason is that it is very easy to use unsafe cryptography functions, which does not benefit the security of your application. In addition, the libm crypt (which the PHP module uses) has not been in development for quite some time (last change in 2008!). Many bugs and safety risks are therefore not resolved. The vrandom generator is also very unreliable, which is very important for secure encryption.

[chrisandy]

Apologies if I'm wrong - I was reading this thread: https://forums.modx.com/thread/104257/alternative-to-filelister-extra

[funkyduke]

the question still remains if there's a good (save) alternative to work with Modx.

[funkyduke]

In meanwhile there's a solution. It was indeed mcrypt. We replaced it with openSSL and everything works fine now! We will do a pull request.

[gkarol]

forked from splittingred/FileLister:

https://github.com/ThijsFeryn/FileLister/tree/mcrypt_to_openssl

[crimsonpixel]

I had this issue today and had to roll back to 7.1. Any chance this could be resolved.

[funkyduke]

I had this issue today and had to roll back to 7.1. Any chance this could be resolved.

Use this solution: https://github.com/ThijsFeryn/FileLister/tree/mcrypt_to_openssl

[crimsonpixel]

Will this solution upgrade FileLister by uploading as a separate package? Don't want to break this particular site if its meant to be applied a different way.

[funkyduke]

It's not a seperate package, only a fix to the existing one. You have to copy it manually.

[crimsonpixel]

Seems when I upgrade to 7.3 the extra breaks again... Nothing in error log to indicate issue it just simply doesn't work.

This fix seemed to correct it 8bc241b