Deploy PHP application

lsolesen · December 9, 2022, 7:44am

I am trying to deploy a PHP-application to Fly.io, but with no luck. I just get the “Hello from Fly”-message.

I used to have a Heroku instance, added the fly.toml and I get no error messages when doing fly deploy.

My repository is here? GitHub - vih/vih-viggo-standalone: Stand alone app for getting calendar items from Viggo

Is it because I am missing a Dockerfile? - and how should the file look to be able to deploy this very simple PHP-app?

shrink · December 9, 2022, 2:01pm

Per the App Configuration documentation, build.image is the builder option to skip the building process by specifying an image that has already been built.

[build]
  image = "flyio/hellofly:latest"

So, in your configuration, Fly is ignoring your application entirely and deploying flyio/hellofly:latest which is the source of the “Hello from Fly” message you’re seeing.

I haven’t used the buildpacks before, but as I understand it from the documentation, your configuration is almost correct. The changes that are required are:

Remove the build.image
Select a php buildpack
Select a builder that is compatible with the php buildpack

I found paketo-buildpacks/php, which says in the README that it’s compatible with 2 builders: paketobuildpacks/builder-jammy-full and paketobuildpacks/builder:full. Unfortunately, PHP 7.4 is at end of life, so it’s not supported by either of these builders: you’ll need to upgrade your app to PHP 8 if you wish to use these builders. The good news is, based on a quick scan of your code; there’s no reason you need PHP 7.4, so if you modify your composer.json to allow for PHP 8, it should work fine:

    "require": {
-        "php": "7.4.*",
+        "php": "~8",

Run composer update php to apply those changes to your composer.lock. Then you can update your fly.toml to use the correct builder and buildpack:

[build]
- builder = "paketobuildpacks/builder:base"
+ builder = "paketobuildpacks/builder:full"
- buildpacks = ["gcr.io/paketo-buildpacks/nodejs"]
+ buildpacks = ["gcr.io/paketo-buildpacks/php"]
- image = "flyio/hellofly:latest"
-  [build.args]
-   PHP_VERSION = "7.4"
-   NODE_VERSION = "14"

You’ll also need to change from port 80 to port 8080. Your final fly.toml will be:

# fly.toml file generated for vih-kalendersiden on 2022-12-09T08:21:47+01:00

app = "vih-kalendersiden"
kill_signal = "SIGINT"
kill_timeout = 5
processes = []

[build]
  builder = "paketobuildpacks/builder-jammy-full"
  buildpacks = ["gcr.io/paketo-buildpacks/php"]

[env]
APP_URL = "https://vih-kalendersiden.fly.dev"
PORT = 8080

[experimental]
  allowed_public_ports = []
  auto_rollback = true

[[services]]
  http_checks = []
  internal_port = 8080
  processes = ["app"]
  protocol = "tcp"
  script_checks = []
  [services.concurrency]
    hard_limit = 25
    soft_limit = 20
    type = "connections"

  [[services.ports]]
    force_https = true
    handlers = ["http"]
    port = 8080

  [[services.ports]]
    handlers = ["tls", "http"]
    port = 443

  [[services.tcp_checks]]
    grace_period = "1s"
    interval = "15s"
    restart_limit = 0
    timeout = "2s"

The final change is to delete the Procfile, then run fly deploy and your application should be good to go I have submitted this all as a Pull Request to your repository: fix: Build application for Fly deployment using buildpack by shrink · Pull Request #9 · vih/vih-viggo-standalone · GitHub

Johannes-Fly · December 9, 2022, 2:40pm

Hi there!

I have checked the solution and can verify it works. Thanks @shrink for contributing!
If you have any other questions, don’t hesitate to reach out.

Johannes

lsolesen · December 10, 2022, 11:10am

Thanks @shrink. It was a really good starting point.

Do you know how I can enable php-extensions in the buildpack? One of my dependencies has some dependencies that is not satisfied.

Paketo Buildpack for Composer Install 0.2.0
  Executing build process
  Building new layer /layers/paketo-buildpacks_composer-install/composer-packages
  Running 'composer install'
exit status 2
    Installing dependencies from lock file
    Verifying lock file contents can be installed on current platform.
    Your lock file does not contain a compatible set of packages. Please run composer update.
    
      Problem 1
        - kigkonsult/icalcreator is locked to version v2.41.71 and an update of this package was not requested.
        - kigkonsult/icalcreator v2.41.71 requires ext-zlib * -> it is missing from your system. Install or enable PHP's zlib extension.
    
    To enable extensions, verify that they are enabled in your .ini files:
        - /layers/paketo-buildpacks_composer-install/composer-php-ini/composer-php.ini
        - /layers/paketo-buildpacks_php-dist/php/etc/buildpack.ini
        - /layers/paketo-buildpacks_php-dist/php/etc/php.ini
        - /layers/paketo-buildpacks_php-dist/php/etc/buildpack.ini
        - /layers/paketo-buildpacks_php-dist/php/etc/php.ini
    You can also run `php --ini` in a terminal to see which files are used by PHP in CLI mode.
    Alternatively, you can run Composer with `--ignore-platform-req=ext-zlib` to temporarily ignore these required extensions.
ERROR: failed to build: exit status 1
Error failed to fetch an image or build from source: executing lifecycle: failed with status code: 51

shrink · December 10, 2022, 1:34pm

Reviewing the documentation, it looks like, unfortunately, zlib isn’t supported within this buildpack and there is no option to install custom extensions. The package you depend on (kigkonsult/icalcreator) didn’t introduce the dependency on zlib until version 2.41.39 (earlier this year), so you could downgrade to 2.41.30. Alternatively, you could bypass the dependency check and cross your fingers that the code you’re relying on doesn’t actually need zlib: I couldn’t find any apparent reason it would be required… to do that you could add the build argument suggested in the error message, e.g:

[build]
  builder = "paketobuildpacks/builder:full"
  buildpacks = ["gcr.io/paketo-buildpacks/php"]
  [build.args]
    BP_COMPOSER_INSTALL_OPTIONS = "--ignore-platform-req=ext-zlib"

If downgrading to version 2.41.30 isn’t an option, and ignoring the dependency on zlib isn’t an option, then I think (without experience of buildpacks) the next best option would be to use your own Dockerfile instead of relying on a buildpack. If you don’t have Docker experience, that might be a painful switch but it would be straight forward if you know what you’re doing with Docker

lsolesen · December 11, 2022, 9:05am

Wow. @shrink thank you so much for your help.

I downgraded the dependency and was able to get it built now.

However, I keep getting this error, which I suppose is openssl not properly setup.

PHP Warning: file_get_contents(): Unable to find the wrapper &quot;https&quot; - did you forget to enable it when you configured PHP? in /workspace/index.php on line 129

I have a ´require` in my composer set to:

    "require": {
        "php": "~8",
        "ext-openssl": "*"
     }

As here: vih-viggo-standalone/composer.json at master · vih/vih-viggo-standalone · GitHub

So it should setup the extension requirement as per How to Build PHP Apps with Paketo Buildpacks - Paketo Buildpacks

But maybe there is something that i do not understand properly.

shrink · December 11, 2022, 1:21pm

According to this comment on GitHub the openssl extension is not automatically available through the composer require behaviour due to an issue that is yet to be fixed. However, according to this issue there is a workaround whereby you create your own .ini file and enable the extension there:

github.com/paketo-buildpacks/php-composer

Buildpack does not detect that openssl is a required extension

opened 12:42AM - 25 Apr 21 UTC

dmikusa

## What happened?  Create and run the sample app as described in this issue. When deployed, it will not have the openssl extension enabled. https://github.com/paketo-buildpacks/php/issues/366#issue-866600764 Based on this, it seems like the auto detection is run but it's missing OpenSSL. It should have been though as when I run `composer check-platform-reqs` locally, it shows openssl as required. The php.ini snippet generated by the buildpack only has: ``` extension = fileinfo.so extension = mbstring.so ``` My theory is that because openssl is a required extension to run Composer, we enable it in the php.ini that's used to execute composer. Thus when `composer check-platform-reqs` executes, it sees openssl as already available. This isn't correct though because it's only available in the limited environment where we execute Composer. I think there's a couple ways that we could fix this: 1. Always enable the openssl extension. It is quite widely used at this point. I think the risk here is that if we enable it, and there's a security vulnerability then an app that doesn't use it could potentially be vulnerable when it otherwise wouldn't be. 2. When we check for extensions with `composer check-platform-reqs`, we normally filter by extensions listed missing. We need to look for both missing or the name openssl. If openssl is listed, it'll never be missing, so we just need to add it to the list of extensions to enable. * What were you attempting to do? Run an app where openssl is a extension required through Composer. * What did you expect to happen? The composer extension to be available. * What was the actual behavior? Please provide log output, if possible. Other extensions were detected correctly, but not openssl. ## Build Configuration * What platform (`pack`, `kpack`, `tekton` buildpacks plugin, etc.) are you using? Please include a version. Pack. * What buildpacks are you using? Please include versions. * What builder are you using? If custom, can you provide the output from `pack inspect-builder <builder>`? full * Can you provide a sample app or relevant configuration (`buildpack.yml`, `nginx.conf`, etc.)? https://github.com/paketo-buildpacks/php/issues/366#issue-866600764 ## Checklist * [x] I have included log output. * [x] The log output includes an error message. * [x] I have included steps for reproduction.

diff --git a/.php.ini.d/extension.ini b/.php.ini.d/extension.ini
new file mode 100644
index 0000000..cb58b9c
--- /dev/null
+++ b/.php.ini.d/extension.ini
@@ -0,0 +1 @@
+extension=openssl.so

I ran a quick test and it seems to work – /calendar/proxy/vih loads, no error!

lsolesen · December 11, 2022, 5:38pm

@shrink. Thank you so much. You totally saved my weekend. Have a really good day.