search

aurelia / polyfills

The minimal set of polyfills needed to run Aurelia.
MIT License
26 stars 27 forks source link

IE 11 issue - still get "Access Denied" in getOwnPropertyNames #44

Closed matik12 closed 7 years ago

matik12 commented 7 years ago

I'm submitting a bug report

Current behavior: After adding Power Bi which creates an iframe underneath, I can still see following errors in console multiple times:

SCRIPT5: Access is denied.
aurelia-polyfills.js (131,7)

PowerBi.png

This is simple repro project using power bi client library. First install dependencies npm i and then run app i.e. gulp watch. PowerBi token can expire, so please obtain new one -> more info in src/powerbi.ts file. Polyfills-Repro.zip

EisenbergEffect commented 7 years ago

@jdanyow Any chance you can look into this?

jdanyow commented 7 years ago

https://github.com/zloirock/core-js/issues/76

jdanyow commented 7 years ago

@EisenbergEffect do you think we should update our polyfills to fix this issue?

https://github.com/zloirock/core-js/issues/76

https://github.com/paulmillr/es6-shim/commit/cee098d1cb451540e1e8f5341180964bbb66c66d

EisenbergEffect commented 7 years ago

Seems like it. Any idea if it's a small fix?

genachka commented 7 years ago

+1 Same as @mikeesouth noted on #22, we're not seeing this in current releases of Chrome or Firefox. Only on IE 11 (we're using 11.0.9600.18617) 100% of the time as well.

Using aurelia-polyfills@1.2.1

Any updates on this?

magnusdanielson commented 7 years ago

I just noticed @doubleaa93 made the PR, I will skip my PR. Sorry for the delay.

jp7677 commented 7 years ago

Thanks a lot @doubleaa93! Any chance to get this PR merged soon?

EisenbergEffect commented 7 years ago

We'll get it out this week :)

jp7677 commented 7 years ago

@EisenbergEffect Thanks a lot!

EisenbergEffect commented 7 years ago

Merged fix. Releasing in a few minutes.