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.