(bug 13453) Fix rebuildrecentchanges for DB's with FK constraints
[lhc/web/wiklou.git] / includes / HttpFunctions.php
index a10f0d4..90845f5 100644 (file)
@@ -7,16 +7,20 @@
  */
 
 class Http {
-       const SYNC_DOWNLOAD = 1;  // syncronys upload (in a single request)
-       const ASYNC_DOWNLOAD = 2; // asynchronous upload we should spawn out another process and monitor progress if possible)
+       const SYNC_DOWNLOAD = 1;  // syncronous upload (in a single request)
+       const ASYNC_DOWNLOAD = 2; // asynchronous upload
 
        var $body = '';
-       public static function request( $url, $opts = array() ) {
-               $req = new HttpRequest( $url, $opts );
+
+       public static function request( $method, $url, $opts = array() ) {
+               $opts['method'] = ( strtoupper( $method ) == 'GET' || strtoupper( $method ) == 'POST' )
+                       ? strtoupper( $method ) : null;
+               $req = HttpRequest::newRequest( $url, $opts );
                $status = $req->doRequest();
-               if( $status->isOK() ){
+               if( $status->isOK() ) {
                        return $status->value;
                } else {
+                       wfDebug( 'http error: ' . $status->getWikiText() );
                        return false;
                }
        }
@@ -24,29 +28,38 @@ class Http {
        /**
         * Simple wrapper for Http::request( 'GET' )
         */
-       public static function get( $url, $opts = array() ) {
-               $opt['method'] = 'GET';
-               return Http::request( $url, $opts );
+       public static function get( $url, $timeout = false, $opts = array() ) {
+               global $wgSyncHTTPTimeout;
+               if( $timeout )
+                       $opts['timeout'] = $timeout;
+               return Http::request( 'GET', $url, $opts );
        }
 
        /**
         * Simple wrapper for Http::request( 'POST' )
         */
        public static function post( $url, $opts = array() ) {
-               $opts['method'] = 'POST';
-               return Http::request( $url, $opts );
+               return Http::request( 'POST', $url, $opts );
        }
 
-       public static function doDownload( $url, $target_file_path , $dl_mode = self::SYNC_DOWNLOAD , $redirectCount = 0 ){
-               global $wgPhpCliPath, $wgMaxUploadSize, $wgMaxRedirects;
+       public static function doDownload( $url, $target_file_path, $dl_mode = self::SYNC_DOWNLOAD,
+               $redirectCount = 0 )
+       {
+               global $wgPhpCli, $wgMaxUploadSize, $wgMaxRedirects;
                // do a quick check to HEAD to insure the file size is not > $wgMaxUploadSize
-               $head = get_headers( $url, 1 );
+               $headRequest = HttpRequest::newRequest( $url, array( 'headers_only' => true ) );
+               $headResponse = $headRequest->doRequest();
+               if( !$headResponse->isOK() ) {
+                       return $headResponse;
+               }
+               $head = $headResponse->value;
 
                // check for redirects:
-               if( isset( $head['Location'] ) && strrpos( $head[0], '302' ) !== false ){
-                       if( $redirectCount < $wgMaxRedirects ){
-                               if( UploadFromUrl::isValidURI( $head['Location'] ) ){
-                                       return self::doDownload( $head['Location'], $target_file_path , $dl_mode, $redirectCount++ );
+               if( isset( $head['Location'] ) && strrpos( $head[0], '302' ) !== false ) {
+                       if( $redirectCount < $wgMaxRedirects ) {
+                               if( self::isValidURI( $head['Location'] ) ) {
+                                       return self::doDownload( $head['Location'], $target_file_path,
+                                               $dl_mode, $redirectCount++ );
                                } else {
                                        return Status::newFatal( 'upload-proto-error' );
                                }
@@ -55,28 +68,30 @@ class Http {
                        }
                }
                // we did not get a 200 ok response:
-               if( strrpos( $head[0], '200 OK' ) === false ){
+               if( strrpos( $head[0], '200 OK' ) === false ) {
                        return Status::newFatal( 'upload-http-error', htmlspecialchars( $head[0] ) );
                }
 
                $content_length = ( isset( $head['Content-Length'] ) ) ? $head['Content-Length'] : null;
-               if( $content_length ){
-                       if( $content_length > $wgMaxUploadSize ){
-                               return Status::newFatal( 'requested file length ' . $content_length . ' is greater than $wgMaxUploadSize: ' . $wgMaxUploadSize );
+               if( $content_length ) {
+                       if( $content_length > $wgMaxUploadSize ) {
+                               return Status::newFatal( 'requested file length ' . $content_length .
+                                       ' is greater than $wgMaxUploadSize: ' . $wgMaxUploadSize );
                        }
                }
 
-               // check if we can find phpCliPath (for doing a background shell request to php to do the download:
-               if( $wgPhpCliPath && wfShellExecEnabled() && $dl_mode == self::ASYNC_DOWNLOAD ){
-                       wfDebug( __METHOD__ . "\ASYNC_DOWNLOAD\n" );
-                       // setup session and shell call:
+               // check if we can find phpCliPath (for doing a background shell request to
+               // php to do the download:
+               if( $wgPhpCli && wfShellExecEnabled() && $dl_mode == self::ASYNC_DOWNLOAD ) {
+                       wfDebug( __METHOD__ . "\nASYNC_DOWNLOAD\n" );
+                       //setup session and shell call:
                        return self::initBackgroundDownload( $url, $target_file_path, $content_length );
-               } else if( $dl_mode == self::SYNC_DOWNLOAD ){
+               } else {
                        wfDebug( __METHOD__ . "\nSYNC_DOWNLOAD\n" );
                        // SYNC_DOWNLOAD download as much as we can in the time we have to execute
                        $opts['method'] = 'GET';
                        $opts['target_file_path'] = $target_file_path;
-                       $req = new HttpRequest( $url, $opts );
+                       $req = HttpRequest::newRequest( $url, $opts );
                        return $req->doRequest();
                }
        }
@@ -86,8 +101,10 @@ class Http {
         * should write to a file location and give updates
         *
         */
-       private function initBackgroundDownload( $url, $target_file_path, $content_length = null ){
-               global $wgMaxUploadSize, $IP, $wgPhpCliPath;
+       private static function initBackgroundDownload( $url, $target_file_path,
+               $content_length = null )
+       {
+               global $wgMaxUploadSize, $IP, $wgPhpCli, $wgServer;
                $status = Status::newGood();
 
                // generate a session id with all the details for the download (pid, target_file_path )
@@ -97,6 +114,8 @@ class Http {
                // store the url and target path:
                $_SESSION['wsDownload'][$upload_session_key]['url'] = $url;
                $_SESSION['wsDownload'][$upload_session_key]['target_file_path'] = $target_file_path;
+               // since we request from the cmd line we lose the original host name pass in the session:
+               $_SESSION['wsDownload'][$upload_session_key]['orgServer'] = $wgServer;
 
                if( $content_length )
                        $_SESSION['wsDownload'][$upload_session_key]['content_length'] = $content_length;
@@ -105,20 +124,22 @@ class Http {
                $_SESSION['wsDownload'][$upload_session_key]['loaded'] = 0;
 
                // run the background download request:
-               $cmd = $wgPhpCliPath . ' ' . $IP . "/maintenance/http_session_download.php --sid {$session_id} --usk {$upload_session_key}";
-               $pid = wfShellBackgroundExec( $cmd, $retval );
+               $cmd = $wgPhpCli . ' ' . $IP . "/maintenance/http_session_download.php " .
+                       "--sid {$session_id} --usk {$upload_session_key} --wiki " . wfWikiId();
+               $pid = wfShellBackgroundExec( $cmd );
                // the pid is not of much use since we won't be visiting this same apache any-time soon.
                if( !$pid )
                        return Status::newFatal( 'could not run background shell exec' );
 
-               // update the status value with the $upload_session_key (for the user to check on the status of the upload)
+               // update the status value with the $upload_session_key (for the user to
+               // check on the status of the upload)
                $status->value = $upload_session_key;
 
                // return good status
                return $status;
        }
 
-       function getUploadSessionKey(){
+       static function getUploadSessionKey() {
                $key = mt_rand( 0, 0x7fffffff );
                $_SESSION['wsUploadData'][$key] = array();
                return $key;
@@ -127,22 +148,26 @@ class Http {
        /**
         * used to run a session based download. Is initiated via the shell.
         *
-        * @param $session_id  String: the session id to grab download details from
+        * @param $session_id String: the session id to grab download details from
         * @param $upload_session_key String: the key of the given upload session
         *  (a given client could have started a few http uploads at once)
         */
-       public static function doSessionIdDownload( $session_id, $upload_session_key ){
-               global $wgUser, $wgEnableWriteAPI, $wgAsyncHTTPTimeout;
-               wfDebug( __METHOD__ . "\n\ndoSessionIdDownload\n\n" );
+       public static function doSessionIdDownload( $session_id, $upload_session_key ) {
+               global $wgUser, $wgEnableWriteAPI, $wgAsyncHTTPTimeout, $wgServer,
+                               $wgSessionsInMemcached, $wgSessionHandler, $wgSessionStarted;
+               wfDebug( __METHOD__ . "\n\n doSessionIdDownload :\n\n" );
                // set session to the provided key:
                session_id( $session_id );
+               //fire up mediaWiki session system:
+               wfSetupSession();
+
                // start the session
-               if( session_start() === false ){
+               if( session_start() === false ) {
                        wfDebug( __METHOD__ . ' could not start session' );
                }
-               //get all the vars we need from session_id
-               if(!isset($_SESSION[ 'wsDownload' ][$upload_session_key])){
-                       wfDebug(  __METHOD__ .' Error:could not find upload session');
+               // get all the vars we need from session_id
+               if( !isset( $_SESSION[ 'wsDownload' ][$upload_session_key] ) ) {
+                       wfDebug(  __METHOD__ . ' Error:could not find upload session');
                        exit();
                }
                // setup the global user from the session key we just inherited
@@ -150,28 +175,38 @@ class Http {
 
                // grab the session data to setup the request:
                $sd =& $_SESSION['wsDownload'][$upload_session_key];
-               // close down the session so we can other http queries can get session updates:
-               session_write_close();
 
-               $req = new HttpRequest( $sd['url'], array(
+               // update the wgServer var ( since cmd line thinks we are localhost
+               // when we are really orgServer)
+               if( isset( $sd['orgServer'] ) && $sd['orgServer'] ) {
+                       $wgServer = $sd['orgServer'];
+               }
+               // close down the session so we can other http queries can get session
+               // updates: (if not $wgSessionsInMemcached)
+               if( !$wgSessionsInMemcached )
+                       session_write_close();
+
+               $req = HttpRequest::newRequest( $sd['url'], array(
                        'target_file_path'  => $sd['target_file_path'],
                        'upload_session_key'=> $upload_session_key,
-                       'timeout'           => $wgAsyncHTTPTimeout
+                       'timeout'           => $wgAsyncHTTPTimeout,
+                       'do_close_session_update' => true
                ) );
                // run the actual request .. (this can take some time)
-               wfDebug( __METHOD__ . "do Request: " . $sd['url'] . ' tf: ' . $sd['target_file_path'] );
+               wfDebug( __METHOD__ . 'do Session Download :: ' . $sd['url'] . ' tf: ' .
+                       $sd['target_file_path'] . "\n\n");
                $status = $req->doRequest();
                //wfDebug("done with req status is: ". $status->isOK(). ' '.$status->getWikiText(). "\n");
 
                // start up the session again:
-               if( session_start() === false ){
+               if( session_start() === false ) {
                        wfDebug( __METHOD__ . ' ERROR:: Could not start session');
                }
                // grab the updated session data pointer
                $sd =& $_SESSION['wsDownload'][$upload_session_key];
                // if error update status:
-               if( !$status->isOK() ){
-                       $sd['apiUploadResult'] = ApiFormatJson::getJsonEncode(
+               if( !$status->isOK() ) {
+                       $sd['apiUploadResult'] = FormatJson::encode(
                                array( 'error' => $status->getWikiText() )
                        );
                }
@@ -179,10 +214,16 @@ class Http {
                if( $status->isOK() ){
                        // setup the FauxRequest
                        $fauxReqData = $sd['mParams'];
+
+                       // Fix boolean parameters
+                       foreach( $fauxReqData as $k => $v ) {
+                               if( $v === false )
+                                       unset( $fauxReqData[$k] );
+                       }
+
                        $fauxReqData['action'] = 'upload';
                        $fauxReqData['format'] = 'json';
                        $fauxReqData['internalhttpsession'] = $upload_session_key;
-
                        // evil but no other clean way about it:
                        $faxReq = new FauxRequest( $fauxReqData, true );
                        $processor = new ApiMain( $faxReq, $wgEnableWriteAPI );
@@ -196,7 +237,6 @@ class Http {
                        $printer->execute();
                        $apiUploadResult = ob_get_clean();
 
-                       wfDebug( __METHOD__ . "\n\n got api result:: $apiUploadResult \n" );
                        // the status updates runner will grab the result form the session:
                        $sd['apiUploadResult'] = $apiUploadResult;
                }
@@ -245,19 +285,56 @@ class Http {
                global $wgVersion;
                return "MediaWiki/$wgVersion";
        }
+
+       /**
+        * Checks that the given URI is a valid one
+        * @param $uri Mixed: URI to check for validity
+        */
+       public static function isValidURI( $uri ){
+               return preg_match(
+                       '/(ftp|http|https):\/\/(\w+:{0,1}\w*@)?(\S+)(:[0-9]+)?(\/|\/([\w#!:.?+=&%@!\-\/]))?/',
+                       $uri,
+                       $matches
+               );
+       }
 }
-class HttpRequest{
+
+class HttpRequest {
        var $target_file_path;
        var $upload_session_key;
-
        function __construct( $url, $opt ){
+
                global $wgSyncHTTPTimeout;
                $this->url = $url;
                // set the timeout to default sync timeout (unless the timeout option is provided)
                $this->timeout = ( isset( $opt['timeout'] ) ) ? $opt['timeout'] : $wgSyncHTTPTimeout;
+               //check special key default
+               if($this->timeout == 'default'){
+                       $opts['timeout'] = $wgSyncHTTPTimeout;
+               }
+
                $this->method = ( isset( $opt['method'] ) ) ? $opt['method'] : 'GET';
-               $this->target_file_path = ( isset( $opt['target_file_path'] ) ) ? $opt['target_file_path'] : false;
-               $this->upload_session_key = ( isset( $opt['upload_session_key'] ) ) ? $opt['upload_session_key'] : false;
+               $this->target_file_path = ( isset( $opt['target_file_path'] ) )
+                       ? $opt['target_file_path'] : false;
+               $this->upload_session_key = ( isset( $opt['upload_session_key'] ) )
+                       ? $opt['upload_session_key'] : false;
+               $this->headers_only = ( isset( $opt['headers_only'] ) ) ? $opt['headers_only'] : false;
+               $this->do_close_session_update = isset( $opt['do_close_session_update'] );
+               $this->postData = isset( $opt['postdata'] ) ? $opt['postdata'] : '';
+
+               $this->ssl_verifyhost = (isset( $opt['ssl_verifyhost'] ))? $opt['ssl_verifyhost']: false;
+
+               $this->cainfo = (isset( $opt['cainfo'] ))? $op['cainfo']: false;
+
+       }
+
+       public static function newRequest($url, $opt){
+               # select the handler (use curl if available)
+               if ( function_exists( 'curl_init' ) ) {
+                       return new curlHttpRequest($url, $opt);
+               } else {
+                       return new phpHttpRequest($url, $opt);
+               }
        }
 
        /**
@@ -269,15 +346,15 @@ class HttpRequest{
         *     'adapter'          => 'curl', 'soket'
         */
        public function doRequest() {
-               # Use curl if available
-               if ( function_exists( 'curl_init' ) ) {
-                       return $this->doCurlReq();
-               } else {
-                       return $this->doPhpReq();
-               }
+               # Make sure we have a valid url
+               if( !Http::isValidURI( $this->url ) )
+                       return Status::newFatal('bad-url');
+               //do the actual request:
+               return $this->doReq();
        }
-
-       private function doCurlReq(){
+}
+class curlHttpRequest extends HttpRequest {
+       public function doReq(){
                global $wgHTTPProxy, $wgTitle;
 
                $status = Status::newGood();
@@ -286,16 +363,29 @@ class HttpRequest{
                // proxy setup:
                if ( Http::isLocalURL( $this->url ) ) {
                        curl_setopt( $c, CURLOPT_PROXY, 'localhost:80' );
-               } else if ( $wgHTTPProxy ) {
+               } elseif ( $wgHTTPProxy ) {
                        curl_setopt( $c, CURLOPT_PROXY, $wgHTTPProxy );
                }
 
                curl_setopt( $c, CURLOPT_TIMEOUT, $this->timeout );
                curl_setopt( $c, CURLOPT_USERAGENT, Http::userAgent() );
 
-               if ( $this->method == 'POST' ) {
+               if($this->ssl_verifyhost)
+                       curl_setopt( $c, CURLOPT_SSL_VERIFYHOST, $this->ssl_verifyhost);
+
+               if($this->cainfo)
+                       curl_setopt( $c, CURLOPT_CAINFO, $this->cainfo);
+
+               if ( $this->headers_only ) {
+                       curl_setopt( $c, CURLOPT_NOBODY, true );
+                       curl_setopt( $c, CURLOPT_HEADER, true );
+               } elseif ( $this->method == 'POST' ) {
                        curl_setopt( $c, CURLOPT_POST, true );
-                       curl_setopt( $c, CURLOPT_POSTFIELDS, '' );
+                       curl_setopt( $c, CURLOPT_POSTFIELDS, $this->postData );
+                       // Suppress 'Expect: 100-continue' header, as some servers
+                       // will reject it with a 417 and Curl won't auto retry
+                       // with HTTP 1.0 fallback
+                       curl_setopt( $c, CURLOPT_HTTPHEADER, array( 'Expect:' ) );
                } else {
                        curl_setopt( $c, CURLOPT_CUSTOMREQUEST, $this->method );
                }
@@ -310,11 +400,15 @@ class HttpRequest{
                }
 
                // set the write back function (if we are writing to a file)
-               if( $this->target_file_path ){
-                       $cwrite = new simpleFileWriter( $this->target_file_path, $this->upload_session_key );
-                       if( !$cwrite->status->isOK() ){
+               if( $this->target_file_path ) {
+                       $cwrite = new simpleFileWriter( $this->target_file_path,
+                               $this->upload_session_key,
+                               $this->do_close_session_update
+                       );
+                       if( !$cwrite->status->isOK() ) {
                                wfDebug( __METHOD__ . "ERROR in setting up simpleFileWriter\n" );
                                $status = $cwrite->status;
+                               return $status;
                        }
                        curl_setopt( $c, CURLOPT_WRITEFUNCTION, array( $cwrite, 'callbackWriteBody' ) );
                }
@@ -334,55 +428,119 @@ class HttpRequest{
                        // do something with curl exec error?
                }
                // if direct request output the results to the stats value:
-               if( !$this->target_file_path && $status->isOK() ){
+               if( !$this->target_file_path && $status->isOK() ) {
                        $status->value = ob_get_contents();
                        ob_end_clean();
                }
                // if we wrote to a target file close up or return error
-               if( $this->target_file_path ){
+               if( $this->target_file_path ) {
                        $cwrite->close();
-                       if( !$cwrite->status->isOK() ){
+                       if( !$cwrite->status->isOK() ) {
                                return $cwrite->status;
                        }
                }
 
-               # Don't return the text of error messages, return false on error
-               $retcode = curl_getinfo( $c, CURLINFO_HTTP_CODE );
-               if ( $retcode != 200 ) {
-                       wfDebug( __METHOD__ . ": HTTP return code $retcode\n" );
-                       $status = Status::newFatal( "HTTP return code $retcode\n" );
-               }
-               # Don't return truncated output
-               $errno = curl_errno( $c );
-               if ( $errno != CURLE_OK ) {
-                       $errstr = curl_error( $c );
-                       wfDebug( __METHOD__ . ": CURL error code $errno: $errstr\n" );
+               if ( $this->headers_only ) {
+                       $headers = explode( "\n", $status->value );
+                       $headerArray = array();
+                       foreach ( $headers as $header ) {
+                               if ( !strlen( trim( $header ) ) )
+                                       continue;
+                               $headerParts = explode( ':', $header, 2 );
+                               if ( count( $headerParts ) == 1 ) {
+                                       $headerArray[] = trim( $header );
+                               } else {
+                                       list( $key, $val ) = $headerParts;
+                                       $headerArray[trim( $key )] = trim( $val );
+                               }
+                       }
+                       $status->value = $headerArray;
+               } else {
+                       # Don't return the text of error messages, return false on error
+                       $retcode = curl_getinfo( $c, CURLINFO_HTTP_CODE );
+                       if ( $retcode != 200 ) {
+                               wfDebug( __METHOD__ . ": HTTP return code $retcode\n" );
+                               $status = Status::newFatal( "HTTP return code $retcode\n" );
+                       }
+                       # Don't return truncated output
+                       $errno = curl_errno( $c );
+                       if ( $errno != CURLE_OK ) {
+                               $errstr = curl_error( $c );
+                               wfDebug( __METHOD__ . ": CURL error code $errno: $errstr\n" );
                                $status = Status::newFatal( " CURL error code $errno: $errstr\n" );
+                       }
                }
-               curl_close( $c );
 
+               curl_close( $c );
                // return the result obj
                return $status;
        }
+}
+class phpHttpRequest extends HttpRequest {
+       public function doReq() {
+               global $wgTitle, $wgHTTPProxy;
+               # Check for php.ini allow_url_fopen
+               if( !ini_get( 'allow_url_fopen' ) ) {
+                       return Status::newFatal( 'allow_url_fopen needs to be enabled for http copy to work' );
+               }
 
-       public function doPhpReq(){
-               #$use file_get_contents...
-               # This doesn't have local fetch capabilities...
+               // start with good status:
+               $status = Status::newGood();
+
+               if ( $this->headers_only ) {
+                       $status->value = get_headers( $this->url, 1 );
+                       return $status;
+               }
+
+               // setup the headers
+               $headers = array( "User-Agent: " . Http::userAgent() );
+               if ( is_object( $wgTitle ) ) {
+                       $headers[] = "Referer: ". $wgTitle->getFullURL();
+               }
 
-               $headers = array( "User-Agent: " . self :: userAgent() );
-               if( strcasecmp( $method, 'post' ) == 0 ) {
+               if( strcasecmp( $this->method, 'post' ) == 0 ) {
                        // Required for HTTP 1.0 POSTs
                        $headers[] = "Content-Length: 0";
                }
-               $opts = array(
+               $fcontext = stream_context_create ( array(
                        'http' => array(
-                               'method' => $method,
+                               'method' => $this->method,
                                'header' => implode( "\r\n", $headers ),
-                               'timeout' => $timeout ) );
-               $ctx = stream_context_create( $opts );
+                               'timeout' => $this->timeout )
+                       )
+               );
+               $fh = fopen( $this->url, "r", false, $fcontext);
 
-               $status->value = file_get_contents( $url, false, $ctx );
-               if( !$status->value ){
+               // set the write back function (if we are writing to a file)
+               if( $this->target_file_path ) {
+                       $cwrite = new simpleFileWriter( $this->target_file_path,
+                               $this->upload_session_key, $this->do_close_session_update );
+                       if( !$cwrite->status->isOK() ) {
+                               wfDebug( __METHOD__ . "ERROR in setting up simpleFileWriter\n" );
+                               $status = $cwrite->status;
+                               return $status;
+                       }
+
+                       // read $fh into the simpleFileWriter (grab in 64K chunks since
+                       // it's likely a ~large~ media file)
+                       while ( !feof( $fh ) ) {
+                               $contents = fread( $fh, 65536 );
+                               $cwrite->callbackWriteBody( $fh, $contents );
+                       }
+                       $cwrite->close();
+                       // check for simpleFileWriter error:
+                       if( !$cwrite->status->isOK() ) {
+                               return $cwrite->status;
+                       }
+               } else {
+                       // read $fh into status->value
+                       $status->value = @stream_get_contents( $fh );
+               }
+               //close the url file wrapper
+               fclose( $fh );
+
+               // check for "false"
+               if( $status->value === false ) {
                        $status->error( 'file_get_contents-failed' );
                }
                return $status;
@@ -399,24 +557,27 @@ class simpleFileWriter {
        var $session_id = null;
        var $session_update_interval = 0; // how often to update the session while downloading
 
-       function simpleFileWriter( $target_file_path, $upload_session_key ){
+       function simpleFileWriter( $target_file_path, $upload_session_key,
+               $do_close_session_update = false )
+       {
                $this->target_file_path = $target_file_path;
                $this->upload_session_key = $upload_session_key;
                $this->status = Status::newGood();
+               $this->do_close_session_update = $do_close_session_update;
                // open the file:
                $this->fp = fopen( $this->target_file_path, 'w' );
-               if( $this->fp === false ){
+               if( $this->fp === false ) {
                        $this->status = Status::newFatal( 'HTTP::could-not-open-file-for-writing' );
                }
                // true start time
                $this->prevTime = time();
        }
 
-       public function callbackWriteBody($ch, $data_packet){
-               global $wgMaxUploadSize;
+       public function callbackWriteBody( $ch, $data_packet ) {
+               global $wgMaxUploadSize, $wgLang;
 
                // write out the content
-               if( fwrite( $this->fp, $data_packet ) === false ){
+               if( fwrite( $this->fp, $data_packet ) === false ) {
                        wfDebug( __METHOD__ ." ::could-not-write-to-file\n" );
                        $this->status = Status::newFatal( 'HTTP::could-not-write-to-file' );
                        return 0;
@@ -426,54 +587,64 @@ class simpleFileWriter {
                clearstatcache();
                $this->current_fsize = filesize( $this->target_file_path );
 
-               if( $this->current_fsize > $wgMaxUploadSize ){
+               if( $this->current_fsize > $wgMaxUploadSize ) {
                        wfDebug( __METHOD__ . " ::http download too large\n" );
-                       $this->status = Status::newFatal( 'HTTP::file-has-grown-beyond-upload-limit-killing: downloaded more than ' .
-                               Language::formatSize( $wgMaxUploadSize ) . ' ' );
+                       $this->status = Status::newFatal( 'HTTP::file-has-grown-beyond-upload-limit-killing: ' .
+                               'downloaded more than ' .
+                               $wgLang->formatSize( $wgMaxUploadSize ) . ' ' );
                        return 0;
                }
-
                // if more than session_update_interval second have passed update_session_progress
-               if( $this->upload_session_key && ( ( time() - $this->prevTime ) > $this->session_update_interval ) ) {
-                       $this->prevTime = time();
-                       $session_status = $this->update_session_progress();
-                       if( !$session_status->isOK() ){
-                               $this->status = $session_status;
-                               wfDebug( __METHOD__ . ' update session failed or was canceled');
-                               return 0;
-                       }
+               if( $this->do_close_session_update && $this->upload_session_key &&
+                       ( ( time() - $this->prevTime ) > $this->session_update_interval ) ) {
+                               $this->prevTime = time();
+                               $session_status = $this->update_session_progress();
+                               if( !$session_status->isOK() ) {
+                                       $this->status = $session_status;
+                                       wfDebug( __METHOD__ . ' update session failed or was canceled');
+                                       return 0;
+                               }
                }
                return strlen( $data_packet );
        }
 
-       public function update_session_progress(){
+       public function update_session_progress() {
+               global $wgSessionsInMemcached;
                $status = Status::newGood();
-               // start the session
-               if( session_start() === false){
-                       wfDebug( __METHOD__ . ' could not start session' );
-                       exit( 0 );
+               // start the session (if necessary)
+               if( !$wgSessionsInMemcached ) {
+                       wfSuppressWarnings();
+                       if( session_start() === false ) {
+                               wfDebug( __METHOD__ . ' could not start session' );
+                               exit( 0 );
+                       }
+                       wfRestoreWarnings();
                }
-               $sd =& $_SESSION['wsDownload'][$this->upload_session_key];
+               $sd =& $_SESSION['wsDownload'][ $this->upload_session_key ];
                // check if the user canceled the request:
-               if( $sd['user_cancel'] == true ){
-                       // kill the download
+               if( isset( $sd['user_cancel'] ) && $sd['user_cancel'] == true ) {
+                       //@@todo kill the download
                        return Status::newFatal( 'user-canceled-request' );
                }
                // update the progress bytes download so far:
                $sd['loaded'] = $this->current_fsize;
-               wfDebug( __METHOD__ . ': set session loaded amount to: ' . $sd['loaded'] . "\n");
+
                // close down the session so we can other http queries can get session updates:
-               session_write_close();
+               if( !$wgSessionsInMemcached )
+                       session_write_close();
+
                return $status;
        }
 
-       public function close(){
+       public function close() {
                // do a final session update:
-               $this->update_session_progress();
+               if( $this->do_close_session_update ) {
+                       $this->update_session_progress();
+               }
                // close up the file handle:
-               if( false === fclose( $this->fp ) ){
+               if( false === fclose( $this->fp ) ) {
                        $this->status = Status::newFatal( 'HTTP::could-not-close-file' );
                }
        }
 
-}
\ No newline at end of file
+}