fix(ajaxObservable): only set default Content-Type header when no bod…

…y is sent (#1830)
ReactiveX · Jul 21, 2016 · 5a895e8 · 5a895e8
1 parent 0035583
commit 5a895e8
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/spec/observables/dom/ajax-spec.ts b/spec/observables/dom/ajax-spec.ts
@@ -141,6 +141,20 @@ describe('Observable.ajax', () => {
     });
   });
 
+  it('should not set default Content-Type header when no body is sent', () => {
+    const obj: Rx.AjaxRequest = {
+      url: '/talk-to-me-goose',
+      method: 'GET'
+    };
+
+    Rx.Observable.ajax(obj).subscribe();
+
+    const request = MockXMLHttpRequest.mostRecent;
+
+    expect(request.url).to.equal('/talk-to-me-goose');
+    expect(request.requestHeaders).to.not.have.keys('Content-Type');
+  });
+
   it('should error if createXHR throws', () => {
     let error;
     const obj = {

diff --git a/src/observable/dom/AjaxObservable.ts b/src/observable/dom/AjaxObservable.ts
@@ -191,7 +191,7 @@ export class AjaxSubscriber<T> extends Subscriber<Event> {
     }
 
     // ensure content type is set
-    if (!('Content-Type' in headers) && !(root.FormData && request.body instanceof root.FormData)) {
+    if (!('Content-Type' in headers) && !(root.FormData && request.body instanceof root.FormData) && typeof request.body !== 'undefined') {
       headers['Content-Type'] = 'application/x-www-form-urlencoded; charset=UTF-8';
     }