Merge branch 'change/11-prevent-multiple-option-requests' of https://github.com/lpouwelse/magento-2-cors-requests into lpouwelse-change/11-prevent-multiple-option-requests

Author: thaddeusmt

Plugin/CorsHeadersPlugin.php

@@ -71,6 +71,16 @@ protected function getEnableAmp()
             \Magento\Store\Model\ScopeInterface::SCOPE_STORE);
     }
 
+    /**
+     * Get the Access-Control-Max-Age
+     * @return string
+     */
+    protected function getMaxAge()
+    {
+        return (int) $this->scopeConfig->getValue('web/corsRequests/max_age',
+            \Magento\Store\Model\ScopeInterface::SCOPE_STORE);
+    }
+
     /**
      * Triggers before original dispatch
      * This method triggers before original \Magento\Webapi\Controller\Rest::dispatch and set version
@@ -92,6 +102,9 @@ public function beforeDispatch(
             if ($this->getEnableAmp()) {
                 $this->response->setHeader('AMP-Access-Control-Allow-Source-Origin', rtrim($originUrl,"/"), true);
             }
+            if ((int)$this->getMaxAge() > 0) {
+                $this->response->setHeader('Access-Control-Max-Age', $this->getMaxAge(), true);
+            }
         }
     }
 

composer.json

@@ -1,5 +1,5 @@
 {
-    "name": "splashlab/magento-2-cors-requests",
+    "name": "lpouwelse/magento-2-cors-requests",
     "description": "Enabling cross-origin resource sharing (CORS) requests to Magento 2 API from configured Origin domain",
     "homepage": "https://github.com/splashlab/magento-2-cors-requests",
     "type": "magento2-module",

etc/adminhtml/system.xml

@@ -25,6 +25,11 @@
                     <source_model>Magento\Config\Model\Config\Source\Yesno</source_model>
                     <comment>Enables AMP-Access-Control-Allow-Source-Origin response header for AMP CORS requests</comment>
                 </field>
+                <field id="max_age" translate="label" type="text" sortOrder="40" showInDefault="1" showInWebsite="1"
+                       showInStore="1">
+                    <label>CORS Control Max age</label>
+                    <comment>Enables Access-Control-Max-Age response header for AMP CORS requests (max age in seconds)</comment>
+                </field>
             </group>
         </section>
     </system>