Merge pull request #107 from creative-commoners/pulls/2.10/guzzle

FIX Update oembed config

_config/oembed.yml

@@ -8,16 +8,6 @@ Only:
 ---
 SilverStripe\Core\Injector\Injector:
   # Configure the CWP proxy if defined
-  Embed\Http\DispatcherInterface:
-    class: Embed\Http\CurlDispatcher
+  Psr\Http\Client\ClientInterface.oembed:
     constructor:
-      config:
-        # CURLOPT_PROXY = 10004
-        10004: '`SS_OUTBOUND_PROXY`'
-        # CURLOPT_PROXYPORT = 59
-        59: '`SS_OUTBOUND_PROXY_PORT`'
-
-  # Provide dispatcher to Embeddable implementations
-  SilverStripe\View\Embed\Embeddable:
-    properties:
-      Dispatcher: '%$Embed\Http\DispatcherInterface'
+      - proxy: '`SS_OUTBOUND_PROXY`:`SS_OUTBOUND_PROXY_PORT`'

composer.json

@@ -9,7 +9,7 @@
     ],
     "require": {
         "php": "^7.3 || ^8.0",
-        "silverstripe/framework": "^4.10",
+        "silverstripe/framework": "^4.11",
         "silverstripe/admin": "^1.10",
         "silverstripe/hybridsessions": "^2",
         "silverstripe/environmentcheck": "^2",

tests/OEmbedTest.php

@@ -0,0 +1,28 @@
+<?php
+
+namespace CWP\Core\Tests;
+
+use Embed\Http\Crawler;
+use GuzzleHttp\Client;
+use SilverStripe\Core\Injector\Injector;
+use SilverStripe\Dev\SapphireTest;
+
+class OEmbedTest extends SapphireTest
+{
+    /**
+     * Ensure that the Psr\Http\Client\ClientInterface.oembed created is a
+     * GuzzleHttp\Client which can have the CWP outband proxy configuration applied to it
+     *
+     * This is to ensure the config in cwp-core/_config/oembed.yml aligns with the
+     * configured ClientInterface in the required version of framework
+     */
+    public function testGuzzleProxyConfig()
+    {
+        $reflClass = new \ReflectionClass(Crawler::class);
+        $reflProperty = $reflClass->getProperty('client');
+        $reflProperty->setAccessible(true);
+        $crawler = Injector::inst()->get(Crawler::class);
+        $client = $reflProperty->getValue($crawler);
+        $this->assertSame(Client::class, get_class($client));
+    }
+}