Ticket #7867 (closed defect: fixed)

Opened 3 months ago

Last modified 5 weeks ago

dojox.form.FileUploader: allow djConfig path to uploader.swf?

Reported by: jburke Owned by: mwilcox
Priority: normal Milestone: tbd
Component: DojoX Form Version: 1.2.0
Severity: normal Keywords:
Cc:

Description

I think there are issues trying to use the uploader.swf from a CDN location. There may be crossdomain.xml issues. Since it is unlikely the CDN will be configured with a proper crossdomain.xml file, it might be worth allowing the configuration of the swf location via:

FileUploader?._swfPath = dojo.config.FileUploaderSwf? dojo.moduleUrl("dojox.form", "resources/uploader.swf")

Or something like that. Issue was raised in this thread: http://www.dojotoolkit.org/forum/dojox-dojox/dojox-support/cant-get-dojox-form-fileuploader-work-dojo-1-2-0-aol-cdn

Attachments

CDN.zip (20.2 kB) - added by mwilcox 2 months ago.
Test Case and Patch for CDN

Change History

Changed 2 months ago by mwilcox

Test Case and Patch for CDN

Changed 2 months ago by mwilcox

I've attached a test case of getting the CDN to work with the FileUploader?.

Currently, as you said James, we need to rely upon a local SWF because of the crossdomain.xml required otherwise.

The attached isn't actually a patch - it's a temporary workaround. include the files/FileUploader.js in a script tag, then after the CDN loads, call doPatch (in dojo.addOnLoad), and the FileUploader? is overwritten. The new FileUploader? allows the usage of the included local SWF file. It's a new SWF, with the Security set to allow all domains.

Changed 2 months ago by mwilcox

  • owner changed from dante to mwilcox

Changed 6 weeks ago by mwilcox

  • status changed from new to closed
  • resolution set to fixed

commit 15927

FileUploader? now accepts a parameter for the SWF path, and will first look in: dojo.config.uploaderPath.

Note the SWF *has* to come from the same domain. Setting up security for it to work from our CDN is not realistic.

Changed 6 weeks ago by peller

@jburke: let me know if you think this is critical enough for pushing to the CDNs in 1.2.3. We still haven't quite cut the release.

@mwilcox: do we have a place to document the new djConfig property?

Changed 5 weeks ago by jburke

peller: I"m a bit late getting through my mail: I do not think it needs to go in 1.2.3 :)

Note: See TracTickets for help on using tickets.