fix: autowiring

Author: Chris53897

README.md

@@ -29,23 +29,25 @@ When a health check failed the HTTP Response status code will be 503. And the JS
     composer require openconext/monitor-bundle
     ```
 
- * Add the bundle to your kernel in `app/AppKernel.php`
+ * If you don't use [Symfony Flex](https://symfony.com/doc/current/setup/flex.html), you must enable the bundle manually in the application:
+
     ```php
-    public function registerBundles()
-    {
-        // ...
-        new OpenConext\MonitorBundle\OpenConextMonitorBundle(),
-    }
-    ```
+    // config/bundles.php
+    // in older Symfony apps, enable the bundle in app/AppKernel.php
+    return [
+    // ...
+     OpenConext\MonitorBundle\OpenConextMonitorBundle::class => ['all' => true],
+    ];
+     ```
    
- * Include the routing configuration in `app/config/routing.yml` by adding:
+ * Include the routing configuration in `config/routes.yml` by adding:
     ```yaml
-    openconext_monitor:
+    open_conext_monitor:
         resource:   "@OpenConextMonitorBundle/Resources/config/routing.yml"
         prefix:     /
      ```
  
- * Add security exceptions in `app/config/security.yml` (if this is required at all)
+ * Add security exceptions in `config/security.yml` (if this is required at all)
     ```yaml
     security:
         firewalls:

src/Controller/HealthController.php

@@ -40,7 +40,7 @@ public function __construct(HealthCheckChain $healthChecker)
         $this->healthChecker = $healthChecker;
     }
 
-    public function healthAction(): JsonResponse
+    public function __invoke(): JsonResponse
     {
         $statusResponse = $this->healthChecker->check();
 

src/Controller/InfoController.php

@@ -66,7 +66,7 @@ public function __construct(
         $this->debuggerEnabled = $debuggerEnabled;
     }
 
-    public function infoAction(): JsonResponse
+    public function __invoke(): JsonResponse
     {
         $info = Information::buildFrom(
             BuildPathFactory::buildFrom($this->buildPath),

src/HealthCheck/DoctrineConnectionHealthCheck.php

@@ -55,7 +55,7 @@ public function check(HealthReportInterface $report): HealthReportInterface
                     $table = reset($tables);
                     // Perform a light-weight query on the chosen table
                     $query = 'SELECT * FROM `%s` LIMIT 1';
-                    $this->entityManager->getConnection()->query(sprintf($query, $table->getName()));
+                    $this->entityManager->getConnection()->executeQuery(sprintf($query, $table->getName()));
                 }
             } catch (Exception $e) {
                 return HealthReport::buildStatusDown('Unable to execute a query on the database.');

src/Resources/config/routing.yml

@@ -2,10 +2,10 @@ monitor.info:
     path: /info
     methods: [GET]
     defaults:
-        _controller: openconext.monitor.controller.info:infoAction
+        _controller: openconext.monitor.controller.info
 
 monitor.health:
     path: /health
     methods: [GET]
     defaults:
-        _controller: openconext.monitor.controller.health:healthAction
+        _controller: openconext.monitor.controller.health

src/Resources/config/services.yml

@@ -6,14 +6,15 @@ services:
             - '%kernel.environment%'
             - '%kernel.debug%'
         public: true
-        tags: ['controller.service_arguments']
+        autowire: true
+        tags: ['controller.service_arguments', 'container.service_subscriber']
 
     openconext.monitor.controller.health:
         class: OpenConext\MonitorBundle\Controller\HealthController
         arguments:
             - '@openconext.monitor.health_check_chain'
         public: true
-        tags: ['controller.service_arguments']
+        tags: ['controller.service_arguments', 'container.service_subscriber']
 
     openconext.monitor.health_check_chain:
         class: OpenConext\MonitorBundle\HealthCheck\HealthCheckChain