Opened 10 years ago
Closed 5 years ago
#11704 closed enhancement (patchwelcome)
Allow dojox.av.FLAudio to be located on another (sub)domain than the page
Reported by: | Thomas Bachem | Owned by: | dylan |
---|---|---|---|
Priority: | low | Milestone: | 1.11 |
Component: | Dojox | Version: | 1.5 |
Keywords: | Cc: | Mike Wilcox | |
Blocked By: | Blocking: |
Description
This is somewhat related to my ticket #11677, which is also about adding cross-domain support for FLAudio.
This time it's about the ExternalInterface? domain policy, which needs a
flash.system.Security.allowDomain("*")
statement to work properly across domains.
I've attached a patch file that is working for us.
Attachments (2)
Change History (9)
Changed 10 years ago by
Attachment: | SecurityAllowDomain.patch added |
---|
comment:1 Changed 10 years ago by
Changed 10 years ago by
Attachment: | AllowScriptAccess.patch added |
---|
comment:2 Changed 10 years ago by
I would be wary of adding the Security.allowDomain("*"), this needs to be vetted to make sure it does not open up inadvertent sharing of data from a good to a bad site.
comment:3 Changed 10 years ago by
Cc: | Mike Wilcox added |
---|---|
Owner: | changed from Adam Peller to Tom Trenka |
comment:4 Changed 9 years ago by
Priority: | high → low |
---|
comment:5 Changed 9 years ago by
Owner: | changed from Tom Trenka to Mike Wilcox |
---|
Mike, maybe you'd like to look at these? Not sure how much we're going to support FLAudio anymore...
comment:6 Changed 5 years ago by
Owner: | changed from Mike Wilcox to dylan |
---|---|
Status: | new → assigned |
comment:7 Changed 5 years ago by
Milestone: | tbd → 1.11 |
---|---|
Resolution: | → patchwelcome |
Status: | assigned → closed |
My apologies again that this one was also ignored for so many years.
Given that flash is quickly being abandoned as something that runs by default on the web, I'm going to mark this as patchwelcome. I know there's a patch. If you want to recreate as a pull request via github following the guidelines at https://github.com/dojo/dojo/blob/master/CONTRIBUTING.md , we'll review and land it.
In addition to the Deft change, you will also need to add
to FLAudio.js, see my second patch file.