Ticket #5434 (closed Bug: fixed)

Opened 4 years ago

Last modified 4 years ago

Access Denied error on IE7 and IPv6

Reported by: damo Owned by: fredck
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

5434.patch (454 bytes) - added by damo 4 years ago.
Proposed fix
5434_2.patch (533 bytes) - added by fredck 4 years ago.

Change History

comment:1 Changed 4 years ago by fredck

  • Keywords IE7 added

Works well with IE8, in all modes.

comment:2 Changed 4 years ago by satya

  • Cc satya_minnekanti@… added

comment:3 Changed 4 years ago by garry.yao

  • Keywords Pending added

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

Changed 4 years ago by damo

Proposed fix

comment:4 Changed 4 years ago by damo

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

comment:5 Changed 4 years ago by fredck

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

  • Status changed from new to assigned
  • Owner set to fredck

comment:7 Changed 4 years ago by damo

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 4 years ago by garry.yao

  • Keywords Review+ added; Review? removed

comment:9 Changed 4 years ago by fredck

  • Status changed from assigned to closed
  • Resolution set to fixed

Fixed with [5407].

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