ENH Use guzzle for oembed

silverstripe · May 11, 2022 · 415d940 · 415d940
1 parent baaa323
commit 415d940
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/_config/oembed.yml b/_config/oembed.yml
@@ -4,6 +4,13 @@ Name: coreoembed
 SilverStripe\Core\Injector\Injector:
   SilverStripe\View\Embed\Embeddable:
     class: SilverStripe\View\Embed\EmbedContainer
+  # Use guzzle instead of the default Embed\Embed curl implementation so that
+  # a proxy url can be set via yml if required
+  Psr\Http\Client\ClientInterface.oembed:
+    class: GuzzleHttp\Client
+  Embed\Http\Crawler:
+    constructor:
+      - '%$Psr\Http\Client\ClientInterface.oembed'
   Embed\Embed:
     constructor:
       - '%$Embed\Http\Crawler'
diff --git a/composer.json b/composer.json
@@ -60,7 +60,8 @@
     },
     "conflict": {
         "egulias/email-validator": "^2",
-        "phpunit/phpunit": "^6 || ^7 || ^8"
+        "phpunit/phpunit": "^6 || ^7 || ^8",
+        "cwp/cwp-core": "<2.11.0"
     },
     "provide": {
         "psr/container-implementation": "1.0.0"