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

Upgrade to php 8.3.0 #292

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM php:8.2.7-fpm-alpine3.18
FROM php:8.3.0-fpm-alpine3.18

LABEL maintainer="Ric Harvey <[email protected]>"

Expand Down Expand Up @@ -76,7 +76,6 @@ RUN apk add --no-cache --virtual .sys-deps \
pecl install -o -f mongodb && \
echo "extension=redis.so" > /usr/local/etc/php/conf.d/redis.ini && \
echo "extension=mongodb.so" > /usr/local/etc/php/conf.d/mongodb.ini && \
echo "zend_extension=xdebug" > /usr/local/etc/php/conf.d/xdebug.ini && \
docker-php-source delete && \
mkdir -p /var/www/app && \
# Install composer and certbot
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ If you have improvements or suggestions please open an issue or pull request on
### Versioning
| Docker Tag | Git Release | Nginx Version | PHP Version | Alpine Version |
|-----|-------|-----|--------|--------|
| latest/![docker version](https://img.shields.io/docker/v/richarvey/nginx-php-fpm?sort=semver)| Main Branch |1.24-r6 | 8.2.7 | 3.18 |
| latest/![docker version](https://img.shields.io/docker/v/richarvey/nginx-php-fpm?sort=semver)| Main Branch |1.24-r6 | 8.3.0 | 3.18 |

For other tags please see: [versioning](https://github.com/richarvey/nginx-php-fpm/blob/main/docs/versioning.md)

Expand Down
6 changes: 6 additions & 0 deletions docs/versioning.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,12 @@ We will use the [semver](http://ricostacruz.com/cheatsheets/semver.html) style n

The latest tag will always follow the main branch in git. the other versions will have releases attached.

#### PHP 8.3.x

| Docker Tag | PHP Version | Nginx Version | Alpine Version | Container Scripts | Notes |
|-----|-------|-----|--------|--------|----------|
| 4.0.0 | 8.3.0 |1.24-r6 | 3.18 | 0.3.17 | upgrade to php 8.3.0 |

#### PHP 8.2.x

| Docker Tag | PHP Version | Nginx Version | Alpine Version | Container Scripts | Notes |
Expand Down
4 changes: 1 addition & 3 deletions scripts/start.sh
Original file line number Diff line number Diff line change
Expand Up @@ -174,11 +174,9 @@ if [[ "$ENABLE_XDEBUG" == "1" ]] ; then
echo "Xdebug already enabled... skipping"
else
echo "zend_extension=$(find /usr/local/lib/php/extensions/ -name xdebug.so)" > $XdebugFile # Note, single arrow to overwrite file.
echo "xdebug.start_with_request=yes" >> $XdebugFile
echo "xdebug.client_host=host.docker.internal" >> $XdebugFile
echo "xdebug.mode=debug" >> $XdebugFile
echo "xdebug.remote_log=/tmp/xdebug.log" >> $XdebugFile
echo "xdebug.remote_autostart=false " >> $XdebugFile # I use the xdebug chrome extension instead of using autostart
echo "xdebug.log=/tmp/xdebug.log" >> $XdebugFile
# NOTE: xdebug.remote_host is not needed here if you set an environment variable in docker-compose like so `- XDEBUG_CONFIG=remote_host=192.168.111.27`.
# you also need to set an env var `- PHP_IDE_CONFIG=serverName=docker`
fi
Expand Down