Beefy Boxes and Bandwidth Generously Provided by pair Networks
XP is just a number

Re: Apache2::UploadProgress & Safari bug status??

by McDarren (Abbot)
on Oct 17, 2012 at 08:11 UTC ( #999480=note: print w/replies, xml ) Need Help??

in reply to Apache2::UploadProgress & Safari bug status??

I recently hit this same problem myself, and found that it not only doesn't work with Safari, but also Chrome.
After much head-banging (and significant help from a co-worker), I did find a work-around.

Edit the file /extras/progress.js, and modify line 85 as follows:

-'GET', url, Boolean(handleUploadProgressResults)); +'GET', url, false);

It appears that the call to handleUploadProgressResults will always return true in Chrome/Safari. No idea why, but modifying (hardcoding) this to false does get things working.


Replies are listed 'Best First'.
Re^2: Apache2::UploadProgress & Safari bug status??
by tlhackque (Sexton) on Feb 11, 2013 at 14:04 UTC
    I don't dispute the observation, but neither the original code nor the patch makes sense. At that line, handleUploadProgressResults is NOT called; it's a reference which will be non-null, hence the Boolean will be true. It's a convoluted way of expressing 'true'. That argument to XMLHttpRequest() *should* be true - false makes the request blocking (synchronous), which is always a bad idea as it can hang the browser. If false makes the code work, something else is broken. Perhaps there is some global variable that should be in a closure. This needs more investigation...

    This communication may not represent my employer's views, if any, on the matters discussed.

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://999480]
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others wandering the Monastery: (7)
As of 2016-10-26 08:24 GMT
Find Nodes?
    Voting Booth?
    How many different varieties (color, size, etc) of socks do you have in your sock drawer?

    Results (337 votes). Check out past polls.