search

sixem / ivfi-php

IVFi is a directory indexer that aims to make it easy to browse and explore web-accessible directories.
https://git.five.sh/ivfi/
Other
108 stars 18 forks source link

[BUG] Deprecated messages #47

Closed V1ck3s closed 5 months ago

V1ck3s commented 11 months ago

Describe the bug Deprecated messages after updating nginx.

To Reproduce If relevant, the steps to reproduce the behavior:

  1. Update nginx with the last version
  2. Deprecated message appear

Expected behavior No deprecated messages

Information If possible, fill out the data:

Errors

Deprecated: Creation of dynamic property Indexer::$processor is deprecated in /var/www/html/indexer.php on line 783
Deprecated: Creation of dynamic property Indexer::$client is deprecated in /var/www/html/indexer.php on line 797
Deprecated: Creation of dynamic property Indexer::$timestamp is deprecated in /var/www/html/indexer.php on line 799
Deprecated: Creation of dynamic property Indexer::$debug is deprecated in /var/www/html/indexer.php on line 800
Deprecated: Creation of dynamic property Indexer::$directorySizes is deprecated in /var/www/html/indexer.php on line 801
Deprecated: Creation of dynamic property Indexer::$filter is deprecated in /var/www/html/indexer.php on line 898
Deprecated: Creation of dynamic property Indexer::$exclude is deprecated in /var/www/html/indexer.php on line 911
Deprecated: Creation of dynamic property Indexer::$format is deprecated in /var/www/html/indexer.php on line 917

Additional context I'm using the standalone php file.

sixem commented 11 months ago

Hi,

I think these appear on newer PHP versions (or FPM). It should be related to an older syntax used in the script which is being deprecated in newer PHP versions.

I'm assuming you also updated PHP with the nginx update? Also, if possible, could you give me your PHP version, just to confirm that you're using a fairly new version?

I'll see if I can get this fixed soon. It should just be a matter of updating the syntax to a newer PHP standard on the affected lines.

V1ck3s commented 11 months ago

Yes, you are right. I have updated a docker image, that's using PHP-FPM 8.2 & Nginx 1.24

V1ck3s commented 8 months ago

Can we have a new release with this fix ? Thank's

sixem commented 8 months ago

Can we have a new release with this fix ? Thank's

I'll see if I can do that soon.

In the meantime, you can also build it yourself by cloning the repository, editing the build.options.json if needed, and building:

git clone https://github.com/sixem/ivfi-php
cd ivfi-php
npm install
npm run build
V1ck3s commented 8 months ago

Can we have a new release with this fix ? Thank's

I'll see if I can do that soon.

In the meantime, you can also build it yourself by cloning the repository, editing the build.options.json if needed, and building:

git clone https://github.com/sixem/ivfi-php
cd ivfi-php
npm install
npm run build

I'm not a JavaScript developer and don't want to install npm, but thanks. It would not be possible to do a workflow for release when there is a commit on master branch ?

sixem commented 5 months ago

I've added a workflow that builds the project and produces a downloadable artifact on any changes now.