Opened 9 years ago

Closed 4 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)

SecurityAllowDomain.patch (577 bytes) - added by Thomas Bachem 9 years ago.
AllowScriptAccess.patch (365 bytes) - added by Thomas Bachem 9 years ago.

Download all attachments as: .zip

Change History (9)

Changed 9 years ago by Thomas Bachem

Attachment: SecurityAllowDomain.patch added

comment:1 Changed 9 years ago by Thomas Bachem

In addition to the Deft change, you will also need to add

allowScriptAccess:"always",

to FLAudio.js, see my second patch file.

Changed 9 years ago by Thomas Bachem

Attachment: AllowScriptAccess.patch added

comment:2 Changed 9 years ago by James Burke

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 9 years ago by Adam Peller

Cc: Mike Wilcox added
Owner: changed from Adam Peller to Tom Trenka

comment:4 Changed 8 years ago by Tom Trenka

Priority: highlow

comment:5 Changed 8 years ago by Tom Trenka

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 4 years ago by dylan

Owner: changed from Mike Wilcox to dylan
Status: newassigned

comment:7 Changed 4 years ago by dylan

Milestone: tbd1.11
Resolution: patchwelcome
Status: assignedclosed

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.

Note: See TracTickets for help on using tickets.