Add Symfony 8, PHP 8.5 support and migrate from deprecated XML config to PHP/YAML

.github/workflows/ci.yml

@@ -15,9 +15,9 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        php: [ 8.1, 8.2, 8.3, 8.4 ]
+        php: [ 8.1, 8.2, 8.3, 8.4, 8.5 ]
         deps: [ highest ]
-        symfony: [ 6.4.*, 7.2.*, 7.3.* ]
+        symfony: [ 6.4.*, 7.2.*, 7.3.*, 8.0.* ]
         include:
           - php: 8.1
             deps: lowest
@@ -27,6 +27,12 @@ jobs:
             symfony: 7.2.*
           - php: 8.1
             symfony: 7.3.*
+          - php: 8.1
+            symfony: 8.0.*
+          - php: 8.2
+            symfony: 8.0.*
+          - php: 8.3
+            symfony: 8.0.*
 
     steps:
       - name: Checkout code

DependencyInjection/Compiler/MailerCompilerPass.php

@@ -29,7 +29,7 @@ public function process(ContainerBuilder $container): void
             throw new \InvalidArgumentException('To enable mail reporting you have to install "symfony/mailer".');
         }
 
-        $loader = new Loader\XmlFileLoader($container, new FileLocator(__DIR__.'/../../Resources/config'));
-        $loader->load('symfony_mailer.xml');
+        $loader = new Loader\PhpFileLoader($container, new FileLocator(__DIR__.'/../../Resources/config'));
+        $loader->load('symfony_mailer.php');
     }
 }

DependencyInjection/LiipMonitorExtension.php

@@ -32,10 +32,11 @@ public function __construct()
      */
     public function load(array $configs, ContainerBuilder $container): void
     {
-        $loader = new Loader\XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
-        $loader->load('runner.xml');
-        $loader->load('helper.xml');
-        $loader->load('commands.xml');
+        $locator = new FileLocator(__DIR__.'/../Resources/config');
+        $loader = new Loader\PhpFileLoader($container, $locator);
+        $loader->load('runner.php');
+        $loader->load('helper.php');
+        $loader->load('commands.php');
 
         $configuration = $this->getConfiguration($configs, $container);
         $config = $this->processConfiguration($configuration, $configs);
@@ -47,7 +48,7 @@ public function load(array $configs, ContainerBuilder $container): void
         if ($config['enable_controller']) {
             $container->setParameter(sprintf('%s.view_template', $this->getAlias()), $config['view_template']);
             $container->setParameter(sprintf('%s.failure_status_code', $this->getAlias()), $config['failure_status_code']);
-            $loader->load('controller.xml');
+            $loader->load('controller.php');
         }
 
         $this->configureMailer($container, $config);
@@ -74,7 +75,7 @@ public function load(array $configs, ContainerBuilder $container): void
                 $this->setParameters($container, $check, $group, $values);
 
                 if (!in_array($check, $checksLoaded)) {
-                    $loader->load('checks/'.$check.'.xml');
+                    $loader->load('checks/'.$check.'.php');
                     $checksLoaded[] = $check;
                 }
             }

Dockerfile

@@ -0,0 +1,13 @@
+FROM php:8.3-cli
+
+RUN apt-get update && apt-get install -y \
+    git \
+    unzip \
+    libzip-dev \
+    && docker-php-ext-install zip \
+    && apt-get clean \
+    && rm -rf /var/lib/apt/lists/*
+
+COPY --from=composer:latest /usr/bin/composer /usr/bin/composer
+
+WORKDIR /app

README.md

@@ -59,7 +59,7 @@ If you want to enable the REST API provided by the bundle then add the following
 
 ```yml
 _monitor:
-    resource: "@LiipMonitorBundle/Resources/config/routing.xml"
+    resource: "@LiipMonitorBundle/Resources/config/routing.yaml"
     prefix: /monitor/health
 ```
 
@@ -488,7 +488,7 @@ liip_monitor:
 ## REST API DOCS ##
 
 For documentation on the REST API see: [http://myproject.org/monitor/health/](http://myproject.org/monitor/health/).
-Don't forget to add the bundle routes in your `routing.xml` file.
+Don't forget to add the bundle routes in your `routing.yaml` file.
 
 
 ## Nagios integration ##

Resources/config/checks/apc_fragmentation.php

@@ -0,0 +1,14 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+return function (ContainerConfigurator $container): void {
+    $container->services()
+        ->set('liip_monitor.check.apc_fragmentation', \Laminas\Diagnostics\Check\ApcFragmentation::class)
+        ->args([
+            '%%liip_monitor.check.apc_fragmentation.warning%%',
+            '%%liip_monitor.check.apc_fragmentation.critical%%',
+        ])
+        ->tag('liip_monitor.check', ['alias' => 'apc_fragmentation'])
+        ->public();
+};

Resources/config/checks/apc_memory.php

@@ -0,0 +1,14 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+return function (ContainerConfigurator $container): void {
+    $container->services()
+        ->set('liip_monitor.check.apc_memory', \Laminas\Diagnostics\Check\ApcMemory::class)
+        ->args([
+            '%%liip_monitor.check.apc_memory.warning%%',
+            '%%liip_monitor.check.apc_memory.critical%%',
+        ])
+        ->tag('liip_monitor.check', ['alias' => 'apc_memory'])
+        ->public();
+};

Resources/config/checks/class_exists.php

@@ -0,0 +1,11 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+return function (ContainerConfigurator $container): void {
+    $container->services()
+        ->set('liip_monitor.check.class_exists', \Laminas\Diagnostics\Check\ClassExists::class)
+        ->args(['%%liip_monitor.check.class_exists%%'])
+        ->tag('liip_monitor.check', ['alias' => 'class_exists'])
+        ->public();
+};

Resources/config/checks/cpu_performance.php

@@ -0,0 +1,11 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+return function (ContainerConfigurator $container): void {
+    $container->services()
+        ->set('liip_monitor.check.cpu_performance', \Laminas\Diagnostics\Check\CpuPerformance::class)
+        ->args(['%%liip_monitor.check.cpu_performance%%'])
+        ->tag('liip_monitor.check', ['alias' => 'cpu_performance'])
+        ->public();
+};

Resources/config/checks/custom_error_pages.php

@@ -0,0 +1,15 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+return function (ContainerConfigurator $container): void {
+    $container->services()
+        ->set('liip_monitor.check.custom_error_pages', \Liip\MonitorBundle\Check\CustomErrorPages::class)
+        ->args([
+            '%%liip_monitor.check.custom_error_pages.error_codes%%',
+            '%%liip_monitor.check.custom_error_pages.path%%',
+            param('kernel.project_dir'),
+        ])
+        ->tag('liip_monitor.check', ['alias' => 'custom_error_pages'])
+        ->public();
+};

Resources/config/checks/disk_usage.php

@@ -0,0 +1,15 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+return function (ContainerConfigurator $container): void {
+    $container->services()
+        ->set('liip_monitor.check.disk_usage', \Laminas\Diagnostics\Check\DiskUsage::class)
+        ->args([
+            '%%liip_monitor.check.disk_usage.warning%%',
+            '%%liip_monitor.check.disk_usage.critical%%',
+            '%%liip_monitor.check.disk_usage.path%%',
+        ])
+        ->tag('liip_monitor.check', ['alias' => 'disk_usage'])
+        ->public();
+};

Resources/config/checks/doctrine_dbal.php

@@ -0,0 +1,14 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+return function (ContainerConfigurator $container): void {
+    $container->services()
+        ->set('liip_monitor.check.doctrine_dbal', \Liip\MonitorBundle\Check\DoctrineDbalCollection::class)
+        ->args([
+            service('doctrine'),
+            '%%liip_monitor.check.doctrine_dbal%%',
+        ])
+        ->tag('liip_monitor.check_collection')
+        ->public();
+};

Resources/config/checks/doctrine_migrations.php

@@ -0,0 +1,14 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+return function (ContainerConfigurator $container): void {
+    $container->services()
+        ->set('liip_monitor.check.doctrine_migrations', \Liip\MonitorBundle\Check\DoctrineMigrationsCollection::class)
+        ->args([
+            service('service_container'),
+            '%%liip_monitor.check.doctrine_migrations%%',
+        ])
+        ->tag('liip_monitor.check_collection')
+        ->public();
+};

Resources/config/checks/doctrine_mongodb.php

@@ -0,0 +1,14 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+return function (ContainerConfigurator $container): void {
+    $container->services()
+        ->set('liip_monitor.check.doctrine_mongodb', \Liip\MonitorBundle\Check\DoctrineMongoDbCollection::class)
+        ->args([
+            service('doctrine_mongodb'),
+            '%%liip_monitor.check.doctrine_mongodb%%',
+        ])
+        ->tag('liip_monitor.check_collection')
+        ->public();
+};

Resources/config/checks/expressions.php

@@ -0,0 +1,11 @@
+<?php
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+return function (ContainerConfigurator $container): void {
+    $container->services()
+        ->set('liip_monitor.check.expressions', \Liip\MonitorBundle\Check\ExpressionCollection::class)
+        ->args(['%%liip_monitor.check.expressions%%'])
+        ->tag('liip_monitor.check_collection')
+        ->public();
+};