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

Switch dockerfile to using php8.2 instead of 8.1 due to Alpine changes #15155

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

uberbrady
Copy link
Collaborator

Snyk wanted to auto-upgrade our version of Alpine which we use for Docker. But that seemed to break the Docker build.

It turns out that Alpine changed the php 8.1 APK to be called php81 instead of php. Switching everything to PHP v8.2 seems to restore the php command and the build works again.

Copy link

what-the-diff bot commented Jul 24, 2024

PR Summary

  • Updated PHP Version
    The component that dictates the PHP version in our software was changed. The previous version was PHP 8.1, but with this update, we're now using the newer PHP 8.2. This enhancement could increase our system's overall operation speed and the tools we use to build our software might work more efficiently.

  • Updated PHP Configuration File
    Further, we made a change to a configuration file (PHP.ini), essential for setting various options for our PHP environment. The path pointing to old PHP configuration (php81/php.ini) has now been updated to correspond to the new PHP version (php82/php.ini). This ensures our software runs smoothly and as intended.

@snipe snipe merged commit 4b906e3 into snyk-fix-067da5564c4d29b025cd356b125e6ad9 Jul 24, 2024
8 of 9 checks passed
@snipe snipe deleted the fix_new_alpine branch July 24, 2024 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants