Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Deprecated messages #47

Closed
V1ck3s opened this issue Nov 26, 2023 · 6 comments
Closed

[BUG] Deprecated messages #47

V1ck3s opened this issue Nov 26, 2023 · 6 comments
Assignees

Comments

@V1ck3s
Copy link

V1ck3s commented Nov 26, 2023

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:

  • nginx version: 1.25.3
  • Browser used: FireFox

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
Copy link
Owner

sixem commented Nov 26, 2023

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
Copy link
Author

V1ck3s commented Nov 26, 2023

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

@V1ck3s
Copy link
Author

V1ck3s commented Mar 2, 2024

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

@sixem
Copy link
Owner

sixem commented Mar 2, 2024

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
Copy link
Author

V1ck3s commented Mar 2, 2024

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
Copy link
Owner

sixem commented May 23, 2024

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

@sixem sixem closed this as completed May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants