Rxjs: Rx.Observable.ajax onProgress only set for Upload

[SntsDev]

Hi there,

I hadn't been able to make progressSubscriber.next to fire on a Rx.Observable.ajax instance, when downloading. Checking the code and the log I found that you set onprogress for the upload but you removed the one for download.

As window.XDomainRequest is only available for IE 8 & 9, then on modern browsers onprogress is not assigned.

https://github.com/ReactiveX/rxjs/blob/7bb8280181bf1d5b0a2d3338ede8bea5e30f64c7/src/observable/dom/AjaxObservable.ts#L332-L336

Did you avoid setting xhr.onprogress = xhrProgress for any specific reason? Or, Should it be there next to xhr.upload.onprogress = xhrProgress ?

Thanks
SntsDev

[PMPepper]

It seems to me that AjaxObservable needs to make a distinction between upload progress and download progress. You can get progress events by adding the line xhr.onprogress = xhrProgress, but you can't really tell the different between upload progress and download progress.

Would it make sense to split the progressSubscriber property into uploadProgressSubscriber and downloadProgressSubscriber? the existing progressSubscriber property could be deprecated, and left with the current behaviour to maintain compatibility.

[ankitbko]

It has been more than a year now. What is status of this?