My CDN gives me my own subdomain for normal http traffic on http://cdn.mysite.example.com, but for SSL they give me https://secure.example.com/mysite. So changing the http to https wouldn't work 100% for me.

I was thinking of providing an additional text box under the check box for enabling https that functions exactly like the CDN Mapping text box. If HTTPS is turned on and this is a secure page, the code will look at this text box instead.

Comments

frankkessler’s picture

Assigned: Unassigned » frankkessler
Status: Active » Needs review
StatusFileSize
new5.12 KB

Here's my first shot at a patch. I added a separate text area for adding https cdn hosts. I also removed the current http to https replacement in the code because you can handle that in this new text area. (This is a 6.x patch as I just don't have the time to install 7.x and work from there.)

wim leers’s picture

Status: Needs review » Needs work

Thanks for the patch!

However, your changes would complicate the most common use case, where the domain names *do* remain the same. So I'm afraid I can't commit this until it takes that fact into account.

wim leers’s picture

#1267902: Allow different CDN domain when using HTTPS is almost the same, and has different code.

bendodd’s picture

Can we mark one as a duplicate and close? Don't mind which. We have have a new patch to post soon.

wim leers’s picture

You can choose :)

bendodd’s picture

Status: Needs work » Closed (duplicate)