Cannot embed YouTube link on CWP

[robbieaverill]

Environment:

• mediumtest-uat on CWP
• SilverStripe 4.1.0
• CWP 2.0-rc3

Steps:

• Go to a CMS page
• Click on "Insert media via URL"
• Paste a link to a YouTube video
• Hit "Add media"

Expected:

• Video will be embedded into TinyMCE content

Actual:

• Network request fails with 503 Service Temporarily Unavailable
• Media dialog shows an error

---

Reported by @emteknetnz via SilverStripe Ops, supposedly works correctly on localhost. This could possibly be an Incapsula issue?

---

Pull requests

• 4.1: FIX Add Embed dispatchers to be configured with injector silverstripe-framework#8192
• 4.1: Cannot embed YouTube link on CWP #2168 silverstripe-asset-admin#793
• CWP 2.0: FIX Add proxy configuration for embedded cURL requests cwp-recipe-cms#8
• 4.2: NEW Shift Embeddable and EmbedResource from asset-admin, lazy load Embed to allow injected dependencies silverstripe-framework#8194
• 4.2: API Deprecate Embeddable and EmbedResource, shift into silverstripe/framework silverstripe-asset-admin#795
• CWP 2.1: FIX Add proxy configuration for embedded cURL requests cwp-recipe-cms#9

[madmatt]

/cc @ishannz who is also affected by this.

[indygriffiths]

When this issue initially was raised with RC's our first guess was that the request isn't being correctly passed through the CWP proxy given the error message.

The docs state that By default, we configure the PHP environment with http_proxy and https_proxy variables which will be automatically picked up by curl so it would be worth checking to see if the framework is making the request by some other means that may not use those variables (does Guzzle?)

[caffeineinc]

I created a pull request on silverstripe-asset-admin which hopefully would address the underlying issue. We'd still need to reference that somewhere in the CWP recipe.

silverstripe/silverstripe-asset-admin#793

[robbieaverill]

All PRs added to issue description

[lukereative]

@robbieaverill All reviewed so should be good for a core committer to merge them all

[robbieaverill]

Thanks @lukereative

[robbieaverill]

All PRs merged, fixes will be available in 4.1.x and 4.2.x (incl. CWP 2.1.0)

[katieelks]

We are getting this issue again, confirmed in a couple of environments. Is this a regression perhaps?

[robbieaverill]

Confirmed the regression with CWP 2.1.0 and CWP 2.1.1-rc2. I'll investigate further today with help from SilverStripe Platform team.

[robbieaverill]

Sorry, I'm mistaken. I can't reproduce this any longer on CWP 2.1.1-rc2.

@katieelks can you please ask one of your devs to add the missing error logging configuration from CWP 2.x (see silverstripe/cwp-core#43) and re-deploy, then try again? This should mean you get some debugging information in Graylog.

[brynwhyman]

For reference: after a few iterations, a fix that was originally applied to config in cwp-recipe-cms was moved to cwp-core to ensure it would be picked up by all CWP sites regardless of other modules that are installed. This was released for CWP 2.1.1 in early September.

See silverstripe/cwp-recipe-cms#11

[sunnysideup]

We have CWP 2.1 installed:

in cwp-recipe-cms:

---
Name: cwpoembedconfig
After: coreoembed
Except:
  environment: dev
Only:
  EnvVarSet: SS_OUTBOUND_PROXY
---
SilverStripe\Core\Injector\Injector:
  # Configure the CWP proxy if defined
  Embed\Http\DispatcherInterface:
    class: Embed\Http\CurlDispatcher
    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'

so:

• not yet in core
• not working ;-( despite config being present

We are on:

cwp/cwp                                                  2.1.1             
cwp/cwp-core                                             2.1.1             
cwp/cwp-pdfexport                                        1.0.1             
cwp/cwp-recipe-cms                                       2.1.1             
cwp/cwp-recipe-core                                      2.1.1 

[robbieaverill]

@sunnysideup just checking you resolved this by copying the config into your project as noted on the other issue?

[sunnysideup]

@robbieaverill - yes, that is now working.

Thank you