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.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | stateless-protocol-d8-2584933-9.patch | 692 bytes | rolfmeijer |
Comments
Comment #2
DeeZone commentedConfirmed, the protocol to the Optimizely CDN server should not be defined.
Comment #3
DeeZone commentedPosted solution to 7.x-2.x branch. Solution will be in next maintenance release.
Thanks dremy!
Comment #4
dremy commentedBeat me to the punch! Thanks DeeZone. Love to see such quick updates getting into a stable.
Comment #5
DeeZone commentedComment #6
rolfmeijer commentedReopened 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.)
Comment #7
rolfmeijer commentedComment #8
rolfmeijer commentedSorry, 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.
Comment #9
rolfmeijer commentedOK, 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.
Comment #10
DeeZone commentedAssigning tz_earl as he's in charge of the D8 branch.
Comment #12
tz_earl commentedFixed using the patch from rolfmeijer.
Thanks also to him for pointing out the trailing spaces, which have been removed.