According to Optimizely's "Implement the Optimizely Snippet" page, the snippet should not include the protocol. So I'll be uploading a patch for the current module, if you choose to use.

Comments

dremy created an issue. See original summary.

DeeZone’s picture

Assigned: Unassigned » DeeZone

Confirmed, the protocol to the Optimizely CDN server should not be defined.

DeeZone’s picture

Status: Active » Fixed

Posted solution to 7.x-2.x branch. Solution will be in next maintenance release.

Thanks dremy!

dremy’s picture

Beat me to the punch! Thanks DeeZone. Love to see such quick updates getting into a stable.

DeeZone’s picture

Status: Fixed » Closed (fixed)
rolfmeijer’s picture

Version: 7.x-2.x-dev » 8.x-2.15-rc1
Status: Closed (fixed) » Needs review

Reopened because the same goes for the D8-version. Patch is attached (no rocket science, but I needed it anyway. I had some serious issues with the protocol.)

rolfmeijer’s picture

StatusFileSize
new6.17 KB
rolfmeijer’s picture

Status: Needs review » Needs work

Sorry, about this, the patch is not correct. Simple as it is, I shouldn’t do this in a hurry…
I will re-roll it later.

rolfmeijer’s picture

Status: Needs work » Needs review
StatusFileSize
new692 bytes

OK, re-rolled.

The reason it wasn’t ok the first time was because of trailing spaces in optimizely.module. Please remove them according to the Drupal coding standards.

DeeZone’s picture

Assigned: DeeZone » tz_earl

Assigning tz_earl as he's in charge of the D8 branch.

  • tz_earl committed eac93cf on 8.x-2.x
    Fixed: Issue #2584933: Optimizely suggests no protocol in HTML snippet
    
tz_earl’s picture

Status: Needs review » Closed (fixed)

Fixed using the patch from rolfmeijer.
Thanks also to him for pointing out the trailing spaces, which have been removed.