-
Notifications
You must be signed in to change notification settings - Fork 401
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Stack overflow in IE11 when using "modern" library #16
Comments
Hi! Thank you for the report. What version of the PubNub SDK is this on? |
I first noticed the issue on 3.5.43. Downloaded the latest (3.5.47) and reproduced it there as well. Call stack shown above is on 3.5.47. |
Ty! I think this gives us details for now to start investigating further. |
I am also having this issue. Any word on what may be going on here? bump |
Hi! Thank you for the error submission. What SDK version are you on? |
You may find it best to use the WEB SDK as a stopgap for now. |
@natebrunson can you provide a code snippet that will help us reproduce the issue? |
I am using modern/pubnub.min.js as of: d023d6f. So let me try simply upgrading and using 3.5.48 and retest. Might be faster than whipping up some test code. |
But it appears that nseegmiller said he tested it as recently as 3.5.47... |
@natebrunson also as @stephenlb mentioned, let us know if the standard web versions exhibit the same issue for you. They are available here: https://github.com/pubnub/javascript/tree/master/web. We'll be taking a look as well within the day, but if you find a workaround within https://github.com/pubnub/javascript/tree/master/web then its status-quo good news in the meantime :) |
@geremyCohen your github profile photo highlights your sense of technology knowledge and experience. |
I just checked and my issue still exists in IE11 with modern, does not exist on IE10 with modern, and does not exist in IE11 with web. Just in case @natebrunson doesn't get you a small example of the modern problem, I'll try to come up with one. Currently it's pretty deeply integrated into our rather complex app, so it won't be trivial to create a simple version. |
@nseegmiller thanks for helping us narrow this down. so it sounds like the web version is the workaround until we can get this straightened out. |
Is the modern version available via CDN? I'm trying to build a codepen to demo the issue, but can't seem to find a direct link. |
Good question! We don't have the modern build on the CDN, as it isn't On Mon, Feb 24, 2014 at 4:11 PM, nseegmiller notifications@github.comwrote:
Cheers! |
OK, I managed to reproduce it in a very simple example. I took the entire modern un-minified pubnub and just pasted it into codepen then added my very simple code at the bottom. This works fine in Chrome and throws the stack overflow in IE11. It is found here: http://codepen.io/nseegmiller/pen/lbIxv Let me know if I can help further. |
try the following: var pubnub = PUBNUB.init({
windowing : 1000, // 1 SECOND
publish_key : 'demo',
subscribe_key : 'demo'
}) |
You can see the updates in the pen I posted above. I altering the object as below (which was slightly different than what you mentioned due to the differences in the web and modern libraries): var pubnubObject = PUBNUB({ Got exactly the same results (stack overflow) in IE11. Feel free to fork my pen and test yourself if that helps. |
Thank you for the result. Must be another challenge in this case. Would be great to find the loop that is occurring on the |
Add ability to perform updates in public repository using `-nodeploy.#` tags.
I've confirmed that this does not happen in IE 10, but I can reproduce it every time in IE11 (Win 8.1) using both the modern/pubnub.min.js and modern/pubnub.js. Here is the call stack from from the non-minified version.
The text was updated successfully, but these errors were encountered: