http://wistia.com/doc/account_management

Provide support of not just the current subdomain.wistia.com but also the more white label approach of creating a CNAME record for subdomain.example.com that points to app.wistia.com.

CommentFileSizeAuthor
#1 media_wistia-cname_support-1569332.patch2.41 KBtech4him

Comments

tech4him’s picture

tech4him commented
Status: Active » Needs review
StatusFileSize
new media_wistia-cname_support-1569332.patch2.41 KB

Here is a patch against 6.x-dev that supports CNAME.

Couple of notes. The current account_subdomain field needs to be completed with the full subdomain (e.g. subdomain.wistia.com). This same field can now accept a cname such as media.example.com as well.

I've added a description to the field on the admin screen to show to two options.

Now, this patch does not change the API url since it will still work fine behind the scenes.

Since we use a CNAME for Wistia, I cannot test the subdomain route. Would love for someone to test that.

Thanks.

traviscarden’s picture

traviscarden commented
Version: 6.x-1.x-dev » 6.x-2.0-beta1
Status: Needs review » Needs work

Thanks for the patch, @tech4him. After a more or less complete rewrite of the module for the 2.x branch, it will no longer work. The subdomain is no longer stored at all, because it's no longer used. We could use a hidden system variable to store your CNAME in, perhaps.