search

TrafeX / docker-php-nginx

Docker image with PHP-FPM 8.3 & Nginx 1.26 on Alpine Linux
https://hub.docker.com/r/trafex/php-nginx
MIT License
1.37k stars 736 forks source link

Deprecated dependency php8-json #89

Closed den-dev-icmy closed 2 years ago

den-dev-icmy commented 2 years ago

php8-json dependency is no needed now. https://github.com/TrafeX/docker-php-nginx/blob/b3ee5d84c0a2bc12ac8180e365f9afca326523e4/Dockerfile#L19

proof: https://www.php.net/manual/en/json.installation.php

As of PHP 8.0.0, the JSON extension is a core PHP extension, so it is always enabled.

TrafeX commented 2 years ago

Hi @den-dev-icmy,

Thank you, you're right! I've removed the json extension from the Dockerfile.