Merge pull request #5 from petfinder-com/bug/unsafe-header-user-agent

remove user-agent header

Author: pftjtn

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@petfinder/petfinder-js",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "Petfinder API client",
   "repository": {
     "type": "git",

package.json

@@ -19,7 +19,7 @@ export class Client {
         this.config = config;
         this.http = axios.create({
             baseURL: config.baseUrl || "https://api.petfinder.com/v2",
-            headers: {"User-Agent": "petfinder-js-sdk/v1.0 (https://github.com/petfinder-com/petfinder-js-sdk)"},
+            headers: {"x-api-sdk": "petfinder-js-sdk/v1.0 (https://github.com/petfinder-com/petfinder-js-sdk)"},
         });
 
         this.http.interceptors.response.use((response: AxiosResponse) => {

src/main.ts

@@ -61,7 +61,7 @@ it("Should be able to get sub clients", () => {
     expect(client.organization).toBeInstanceOf(Organization);
 });
 
-it("Should include an identifiable user agent", async () => {
+it("Should include an identifiable header", async () => {
     const mock = new MockAdapter(axios);
     mock.onGet("/").replyOnce(200);
 
@@ -70,6 +70,6 @@ it("Should include an identifiable user agent", async () => {
 
     expect(response.status).toEqual(200);
     expect(mock.history.get.length).toEqual(1);
-    expect(mock.history.get[0].headers["User-Agent"])
+    expect(mock.history.get[0].headers["x-api-sdk"])
         .toEqual("petfinder-js-sdk/v1.0 (https://github.com/petfinder-com/petfinder-js-sdk)");
 });