Merge pull request #638 from 94noni/ttl-header-cache-control

feat: allow to configure header name for reverse_proxy_ttl specific value

Author: dbu

CHANGELOG.md

@@ -4,6 +4,13 @@ Changelog
 3.x
 ===
 
+3.1.0
+-----
+
+### Added
+
+* New Feature: allow configuring the TTL header name for the special `reverse_proxy_ttl` config value. #638
+
 3.0.2
 -----
 

Resources/doc/reference/configuration/headers.rst

@@ -345,3 +345,26 @@ section:
                             s_maxage: 60
 
 This example adds the header ``X-Reverse-Proxy-TTL: 3600`` to your responses.
+
+``ttl_header``
+--------------
+
+**type**: ``string`` default `X-Reverse-Proxy-TTL`
+
+Change the name for the header for reverse proxy.
+
+.. code-block:: yaml
+
+    # app/config/config.yml
+    fos_http_cache:
+        cache_control:
+            ttl_header: X-My-Cache-Control
+            rules:
+                -
+                    headers:
+                        reverse_proxy_ttl: 3600
+                        cache_control:
+                            public: true
+                            s_maxage: 60
+
+This example adds the header ``X-My-Cache-Control: 3600`` to your responses.

src/DependencyInjection/Configuration.php

@@ -277,6 +277,10 @@ private function addCacheControlSection(ArrayNodeDefinition $rootNode): void
                                 ->end()
                             ->end()
                         ->end()
+                        ->scalarNode('ttl_header')
+                            ->defaultValue('X-Reverse-Proxy-TTL')
+                            ->info('Specify the header name to use with the cache_control.reverse_proxy_ttl setting')
+                        ->end()
                         ->arrayNode('rules')
                             ->prototype('array')
                                 ->children();
@@ -330,7 +334,7 @@ private function addCacheControlSection(ArrayNodeDefinition $rootNode): void
                     ->end()
                     ->scalarNode('reverse_proxy_ttl')
                         ->defaultNull()
-                        ->info('Specify an X-Reverse-Proxy-TTL header with a time in seconds for a caching proxy under your control.')
+                        ->info('Specify a custom time to live in seconds for your caching proxy. This value is sent in the custom header configured in cache_control.ttl_header.')
                     ->end()
                     ->arrayNode('vary')
                         ->beforeNormalization()->ifString()->then(function ($v) {

src/DependencyInjection/FOSHttpCacheExtension.php

@@ -48,7 +48,9 @@ public function load(array $configs, ContainerBuilder $container): void
 
         if ($config['debug']['enabled'] || (!empty($config['cache_control']))) {
             $debugHeader = $config['debug']['enabled'] ? $config['debug']['header'] : false;
+            $ttlHeader = $config['cache_control']['ttl_header'];
             $container->setParameter('fos_http_cache.debug_header', $debugHeader);
+            $container->setParameter('fos_http_cache.ttl_header', $ttlHeader);
             $loader->load('cache_control_listener.xml');
         }
 

src/EventListener/CacheControlListener.php

@@ -56,6 +56,7 @@ public function __construct(
          * @var string|false Name of the header or false to add no header
          */
         private readonly string|false $debugHeader = false,
+        private readonly string $ttlHeader = 'X-Reverse-Proxy-TTL',
     ) {
     }
 
@@ -115,9 +116,9 @@ public function onKernelResponse(ResponseEvent $event): void
 
         if (array_key_exists('reverse_proxy_ttl', $options)
             && null !== $options['reverse_proxy_ttl']
-            && !$response->headers->has('X-Reverse-Proxy-TTL')
+            && !$response->headers->has($this->ttlHeader)
         ) {
-            $response->headers->set('X-Reverse-Proxy-TTL', $options['reverse_proxy_ttl'], false);
+            $response->headers->set($this->ttlHeader, $options['reverse_proxy_ttl'], false);
         }
 
         if (!empty($options['vary'])) {

src/Resources/config/cache_control_listener.xml

@@ -9,6 +9,7 @@
                  class="FOS\HttpCacheBundle\EventListener\CacheControlListener"
                  public="true">
             <argument>%fos_http_cache.debug_header%</argument>
+            <argument>%fos_http_cache.ttl_header%</argument>
             <tag name="kernel.event_subscriber" />
         </service>
         <service id="FOS\HttpCacheBundle\EventListener\CacheControlListener" alias="fos_http_cache.event_listener.cache_control" public="true"/>

tests/Resources/Fixtures/config/etag_true.php

@@ -21,6 +21,7 @@
                     ],
                 ],
             ],
+            'ttl_header' => 'X-Reverse-Proxy-TTL',
         ],
     ]
 );

tests/Resources/Fixtures/config/etag_true.xml

@@ -9,6 +9,7 @@
                 <headers etag="true">
                 </headers>
             </rule>
+            <ttl-header>X-Reverse-Proxy-TTL</ttl-header>
         </cache-control>
     </config>
 

tests/Resources/Fixtures/config/etag_true.yml

@@ -5,4 +5,5 @@ fos_http_cache:
                 match:
                     path: null
                 headers:
-                    etag: true
+                    etag: true
+        ttl_header: X-Reverse-Proxy-TTL

tests/Resources/Fixtures/config/etag_weak.php

@@ -21,6 +21,7 @@
                     ],
                 ],
             ],
+            'ttl_header' => 'X-Reverse-Proxy-TTL',
         ],
     ]
 );