make ravenjs version configurable, closes #61

Author: markushausammann

Module.php

@@ -29,6 +29,8 @@
  */
 class Module
 {
+    public const RAVENJS_VERSION = '3.27.0';
+
     /**
      * Translates Zend Framework log levels to Raven log levels.
      */
@@ -232,10 +234,13 @@ protected function setupJavascriptLogging(MvcEvent $event)
     {
         /** @var HeadScript $headScript */
         $headScript    = $event->getApplication()->getServiceManager()->get('ViewHelperManager')->get('headscript');
-        $useRavenjsCDN = $this->config['zend-sentry']['use-ravenjs-cdn'];
 
-        if (!isset($useRavenjsCDN) || $useRavenjsCDN) {
-            $headScript->offsetSetFile(0, '//cdn.ravenjs.com/3.26.2/raven.min.js');
+        $useRavenjsCDN = $this->config['zend-sentry']['use-ravenjs-cdn'] ?? false;
+
+        if ($useRavenjsCDN) {
+            $ravenjsVersion = $this->config['zend-sentry']['ravenjs-version'] ?? self::RAVENJS_VERSION;
+            $cdnUri = sprintf('//cdn.ravenjs.com/%s/raven.min.js', $ravenjsVersion);
+            $headScript->offsetSetFile(0, $cdnUri);
         }
 
         $publicApiKey  = $this->convertKeyToPublic($this->config['zend-sentry']['sentry-api-key']);

README.md

@@ -238,6 +238,12 @@ Just for the record, a copy of the actual global configuration options:
      * If you set this to false you'll need to make sure to load raven-js some other way.
      */
     'use-ravenjs-cdn' => true,
+    
+    /**
+     * Change the raven-js version loaded via CDN if you need to downgrade or we're lagging behind with updating.
+     * No BC breaks, ZendSentry will set the version if your config is missing the key.
+     */
+    'ravenjs-version' => '3.27.0',
 
     /**
      * Set raven config options for the getsentry/sentry-php package here.

config/zend-sentry.global.php.dist

@@ -5,7 +5,7 @@
  * If you have a ./config/autoload/ directory set up for your project, you can
  * drop this config file in it, remove the .dist extension add your configuration details.
  */
-$settings = array(
+$settings = [
     /**
      * Turn ZendSentry off or on as a whole package
      */
@@ -73,22 +73,28 @@ $settings = array(
      */
     'use-ravenjs-cdn' => true,
 
+    /**
+     * Change the raven-js version loaded via CDN if you need to downgrade or we're lagging behind with updating.
+     * No BC breaks, ZendSentry will set the version if your config is missing the key.
+     */
+    'ravenjs-version' => '3.27.0',
+
     /**
      * Set raven config options for the getsentry/sentry-php package here.
      * Raven has sensible defaults set in Raven_Client, if you need to override them, this is where you can do it.
      */
-    'raven-config' => array(),
+    'raven-config' => [],
 
     /**
      * Set ravenjs config options for the getsentry/raven-js package here.
      * This will be json encoded and passed to raven-js when doing Raven.install().
      */
-    'ravenjs-config' => array(),
-);
+    'ravenjs-config' => [],
+];
 
 /**
  * You do not need to edit below this line
  */
-return array(
+return [
     'zend-sentry' => $settings,
-);
+];