Can see counter on /api/download-counter #10837

Author: PowerKiKi

config/autoload/dependencies.global.php

@@ -17,6 +17,8 @@
         // class name.
         'invokables' => [
             Doctrine\ORM\Mapping\UnderscoreNamingStrategy::class => Doctrine\ORM\Mapping\UnderscoreNamingStrategy::class,
+            Application\Handler\DownloadFile::class => Application\Handler\DownloadFile::class,
+            Application\Handler\DownloadCounter::class => Application\Handler\DownloadCounter::class,
         ],
         // Use 'factories' for services provided by callbacks/factory classes.
         'factories' => [

config/routes.php

@@ -57,6 +57,14 @@
         \Application\Handler\DatatransHandler::class,
     ], 'datatrans');
 
+    $app->get('/api/download-file', [
+        \Application\Handler\DownloadFile::class,
+    ]);
+
+    $app->get('/api/download-counter', [
+        \Application\Handler\DownloadCounter::class,
+    ]);
+
     $app->get('/sitemap.xml', [
         SitemapHandler::class,
     ], 'sitemap');

package.json

@@ -6,7 +6,7 @@
         "ng": "ng",
         "postinstall": "playwright install chromium",
         "client-config": "`which php8.2 || which php` ./bin/generate-client-configuration.php",
-        "prerequisite": "mkdir -p htdocs/ && cp -v server/index.php htdocs/ && cp -v server/download-file.php htdocs/ && yarn client-config && yarn codegen",
+        "prerequisite": "mkdir -p htdocs/ && cp -v server/index.php htdocs/ && yarn client-config && yarn codegen",
         "dev": "yarn prerequisite && ng serve",
         "prod": "yarn prerequisite && ng build && find data/tmp/build/browser -type f -not -name '*.gz' -exec gzip --keep --best {} \\; && rm -rf htdocs/ && mv -v data/tmp/build/browser htdocs",
         "test": "yarn prerequisite && ng test",

server/Application/Handler/DownloadCounter.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Application\Handler;
+
+use Ecodev\Felix\Handler\AbstractHandler;
+use Laminas\Diactoros\Response\TextResponse;
+use Psr\Http\Message\ResponseInterface;
+use Psr\Http\Message\ServerRequestInterface;
+
+class DownloadCounter extends AbstractHandler
+{
+    public function handle(ServerRequestInterface $request): ResponseInterface
+    {
+        $path = DownloadFile::COUNTER_PATH;
+        if (!is_readable($path)) {
+            return $this->createError('File not found on disk, or not readable');
+        }
+
+        $response = new TextResponse(file_get_contents($path));
+
+        return $response;
+    }
+}

server/Application/Handler/DownloadFile.php

@@ -0,0 +1,52 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Application\Handler;
+
+use Ecodev\Felix\Handler\AbstractHandler;
+use Laminas\Diactoros\Response;
+use Psr\Http\Message\ResponseInterface;
+use Psr\Http\Message\ServerRequestInterface;
+
+class DownloadFile extends AbstractHandler
+{
+    public const COUNTER_PATH = 'data/download-file-counter/counter.txt';
+
+    public function handle(ServerRequestInterface $request): ResponseInterface
+    {
+        $path = 'data/download-file-counter/file.pdf';
+        $cookie_name = 'artisans_pdf_download_1';
+
+        // Increment counter if no cookie = first visit
+        if (!isset($_COOKIE[$cookie_name])) {
+            $download_count = 0;
+            if (file_exists(self::COUNTER_PATH)) {
+                $download_count = (int) (file_get_contents(self::COUNTER_PATH));
+            }
+            ++$download_count;
+            file_put_contents(self::COUNTER_PATH, $download_count);
+        }
+
+        // Flag cookie
+        setcookie($cookie_name, 'true', time() + (86400 * 60), '/'); // 60 days
+
+        if (!is_readable($path)) {
+            return $this->createError('File not found on disk, or not readable');
+        }
+
+        $resource = fopen($path, 'rb');
+        if ($resource === false) {
+            return $this->createError('Cannot open file on disk');
+        }
+
+        $size = filesize($path);
+        $response = new Response($resource, 200, [
+            'content-type' => 'application/pdf',
+            'Content-Disposition' => 'attachment; filename="' . basename($path) . '"',
+            'content-length' => $size,
+        ]);
+
+        return $response;
+    }
+}

server/download-file.php

@@ -1,25 +1,3 @@
 <?php
 
 declare(strict_types=1);
-$pdf_file = '../data/download-file-counter/file.pdf';
-$counter_path = '../data/download-file-counter/counter.txt';
-$cookie_name = 'artisans_pdf_download_1';
-
-// Increment counter if no cookie = first visit
-if (!isset($_COOKIE[$cookie_name])) {
-    $download_count = 0;
-    if (file_exists($counter_path)) {
-        $download_count = (int) (file_get_contents($counter_path));
-    }
-    ++$download_count;
-    file_put_contents($counter_path, $download_count);
-}
-
-// Flag cookie
-setcookie($cookie_name, 'true', time() + (86400 * 60), '/'); // 60 days
-
-// Do it
-header('Content-Type: application/pdf');
-header('Content-Disposition: attachment; filename="' . basename($pdf_file) . '"');
-readfile($pdf_file);
-exit;