Reports of stalling in Safari about asap HOT 7 CLOSED

I added some internal logging to Q while debugging, where it would log each 'thenable' task and each executed one.

Before the first time a page load is attempted, there are 2 more thenables than tasks run, so it looks like it does stall at some point.

Also, I moved the declaration of flushing to outside the nextTick function, so that it is accessible on the module level, and added Q.isFlushing = function(){return flushing;}

Then when I reproduced the issue again, was able to check and verify - it was stuck flushing. After all tasks in Q have completed, flushing should be false.

I also tried to log the rejections but there weren't any to speak of- at least none that I could successfully log.

I'm in Tel Aviv, Israel, so I would be 10 hours ahead of your time.

i would be more than happy to let you connect to me via TeamViewer or something similar so that you can see the issue reproduced (it's just a difficult scenario to reproduce when you try.. much easier to reproduce when you don't want it to happen ;) ).

i'll check back here tomorrow morning, so if you're interested, I'll stay late to connect with you due to the timezone differences.

-Etai

from asap.

@dtpx No need to stay late. At some point, I’ll either try to reproduce or send you a patch that might address the issue for you to verify.

from asap.

I think the fix is pending at https://github.com/kriskowal/asap/pull/22/files#diff-eb6974d7d0aacfb5eeba15f92952d63cR109

from asap.

@rkatic, I'm making the changes in Q to reflect that pull req. and testing, I'll let you know if this addresses the issue.

from asap.

It's not fixing the issue. I'm still able to get to a state where Q stalls.

I'm trying to understand how this can possibly happen and why only in safari..

Is there a way I can see the actual promises in the queue?

I'm currently going to try to make the tasklist doubly linked, and expose it so that I can inspect it when this happens... if you have a better idea for how to debug it I'm open to more suggestions.

from asap.

any news on this issue?

from asap.

See: #41

from asap.