Opened 8 years ago

Closed 7 years ago

#5434 closed Bug (fixed)

Access Denied error on IE7 and IPv6

Reported by: Damian Owned by: Frederico Caldeira Knabben
Priority: Normal Milestone: CKEditor 3.3
Component: General Version:
Keywords: IE7 IBM Review+ Cc: satya_minnekanti@…

Description

We are testing the editor in an IPv6 environment on IE7.

Script debugger reports the error at:

return i&&new g(i.contentWindow.document);

in ckditor.js. This looks to be in issue in element.js.

I suspect it is the call to contentWindow.document that is causing the "Access Denied" error.

The editor loads, but is not usable.

Attachments (2)

5434.patch (454 bytes) - added by Damian 8 years ago.
Proposed fix
5434_2.patch (533 bytes) - added by Frederico Caldeira Knabben 8 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: IE7 added

Works well with IE8, in all modes.

comment:2 Changed 8 years ago by Satya Minnekanti

Cc: satya_minnekanti@… added

comment:3 Changed 8 years ago by Garry Yao

Keywords: Pending added

WFM in IE7, with both IP and normal domain name.

Changed 8 years ago by Damian

Attachment: 5434.patch added

Proposed fix

comment:4 Changed 8 years ago by Damian

The added patch indicates the problem.

In IE7 document.domain = "[fe80::fe80:fe80:fe80:fe80]"
and window.location.hostname = "fe80::fe80:fe80:fe80:fe80" (i.e. no brackets [] )

This difference is causing isCustomDomain() to return true when in fact it should return false.

Changed 8 years ago by Frederico Caldeira Knabben

Attachment: 5434_2.patch added

comment:5 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review? added; Pending removed

@damo, wonderful job. thanks for the patch.

I've provided a new patch, based on your code, which should produce a more compact output. Please give us the last ok for it, so it'll get approved and committed. Thanks!

comment:6 Changed 7 years ago by Frederico Caldeira Knabben

Owner: set to Frederico Caldeira Knabben
Status: newassigned

comment:7 Changed 7 years ago by Damian

Thanks Fred! The proposed fix works.

One side note here is that the user must add the host to their list of trusted sites in IE otherwise the "Access Denied" error will still occur. There doesn't seem to be any way around this.

comment:8 Changed 7 years ago by Garry Yao

Keywords: Review+ added; Review? removed

comment:9 Changed 7 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: assignedclosed

Fixed with [5407].

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy