Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#1422 closed defect (fixed)

Mixed Content Error from hosteve_browser.js with IE 6.0 when using https

Reported by: guest Owned by: Tom Trenka
Priority: high Milestone:
Component: Core Version: 0.2
Keywords: Cc:
Blocked By: Blocking:

Description

There are many similar reports of problems with mixed content but this one does not seem to be addressed. Also see tickets 1241 548.

The code in question makes sure that dojo gets intialized after the page has loaded and all other libraries have been loaded by using a scirpt tag.

if(dojo.render.html.ie && dojo.render.os.win){
   document.write("<scr"+"ipt id=__ie_onload defer src=javascript:void(0)></scr"+"ipt>");
    var script = document.getElementById("__ie_onload");
    script.onreadystatechange = function() {
      script = document.getElementById("__ie_onload");
      script.onreadystatechange = function() {
         if (this.readyState == "complete") {
             dj_load_init(); // call the onload handler
         }
    };

This code will cause a mixed content error under IE 6.0 when https is used. The attached patch will load without any mixed content warnings though it might not be the most elegant solution.

Attachments (4)

ticket1422_patch.2 (1.2 KB) - added by guest 13 years ago.
Patch for this issue.
ticket1422_patch.3 (1.2 KB) - added by guest 13 years ago.
Patch for this issue.
ticket1422_patch (1.2 KB) - added by guest 13 years ago.
Patch for this issue.
t1422.patch (1.3 KB) - added by guest 13 years ago.
The final final patch

Download all attachments as: .zip

Change History (8)

Changed 13 years ago by guest

Attachment: ticket1422_patch.2 added

Patch for this issue.

Changed 13 years ago by guest

Attachment: ticket1422_patch.3 added

Patch for this issue.

Changed 13 years ago by guest

Attachment: ticket1422_patch added

Patch for this issue.

Changed 13 years ago by guest

Attachment: t1422.patch added

The final final patch

comment:1 Changed 13 years ago by dylan

Component: GeneralCore
Milestone: 0.4
Owner: changed from anonymous to alex

comment:2 Changed 13 years ago by Tom Trenka

Owner: changed from alex to Tom Trenka

comment:3 Changed 13 years ago by Tom Trenka

Resolution: fixed
Status: newclosed

(In [5600]) Fixes #1422 by killing Dean Edwards' IE script tag hack and simply using document.onreadystatechange, which fires before onload.

comment:4 Changed 12 years ago by (none)

Milestone: 0.4

Milestone 0.4 deleted

Note: See TracTickets for help on using tickets.