Merge pull request #31 from uphold/support/remove-user-agent

Remove user-agent default header

Author: ruifreitas

docs/client.md

@@ -35,9 +35,7 @@ export default class extends SDK {
 
 ```json
 {
-  "defaultHeaders": {
-    "user-agent": "uphold-sdk-javascript/<version>"
-  }
+  "defaultHeaders": {}
 }
 ```
 

src/core/services/client.js

@@ -1,7 +1,5 @@
-import { name, version } from '../../../package.json';
-
 export default class Client {
   constructor() {
-    this.defaultHeaders = { 'user-agent': `${name}/${version}` };
+    this.defaultHeaders = {};
   }
 }

test/browser/services/fetch-client.spec.js

@@ -1,5 +1,4 @@
 import { FetchClient, NotFoundError, UnavailableError } from '../../../src/browser';
-import { version } from '../../../package.json';
 import fetchMock from 'fetch-mock';
 
 describe('FetchClient', () => {
@@ -19,15 +18,6 @@ describe('FetchClient', () => {
         });
     });
 
-    it('should send a `User-Agent` header', () => {
-      fetchMock.mock('foo', {});
-
-      return client.request('foo')
-        .then(() => {
-          expect(fetchMock.lastOptions().headers.get('user-agent')).toBe(`@uphold/uphold-sdk-javascript/${version}`);
-        });
-    });
-
     it('should send given `body` and `headers`', () => {
       fetchMock.mock('foo', {});
 

test/core/services/client.spec.js

@@ -1,10 +1,9 @@
 import { Client } from '../../../src/core';
-import { name, version } from '../../../package.json';
 
 describe('Client', () => {
   describe('constructor()', () => {
     it('should set `defaultHeaders` property', () => {
-      expect(new Client().defaultHeaders).toEqual({ 'user-agent': `${name}/${version}` });
+      expect(new Client().defaultHeaders).toEqual({});
     });
   });
 });