Fix: remove duplicate content type header (#537)

* dispatch thrown response

* remove content type header

* add comment on the reliance on the graph client

src/app/services/actions/query-action-creator-util.ts

@@ -24,12 +24,9 @@ export async function anonymousRequest(dispatch: Function, query: IQuery) {
   const escapedUrl = encodeURIComponent(query.sampleUrl);
   const graphUrl = `https://proxy.apisandbox.msdn.microsoft.com/svc?url=${escapedUrl}`;
   const sampleHeaders: any = {};
-
-  if (query.sampleHeaders) {
+  if (query.sampleHeaders && query.sampleHeaders.length > 0) {
     query.sampleHeaders.forEach(header => {
-      if (header.name !== '' && header.value !== '') {
-        sampleHeaders[header.name] = header.value;
-      }
+      sampleHeaders[header.name] = header.value;
     });
   }
 
@@ -101,11 +98,14 @@ const makeRequest = (httpVerb: string, scopes: string[]): Function => {
     const sampleHeaders: any = {};
     sampleHeaders.SdkVersion = 'GraphExplorer/4.0';
 
-    if (query.sampleHeaders) {
+    if (query.sampleHeaders && query.sampleHeaders.length > 0) {
       query.sampleHeaders.forEach(header => {
-        if (header.name !== '' && header.value !== '') {
-          sampleHeaders[header.name] = header.value;
+
+        // We are relying on the graph client to set the content type header.
+        if (header.name.toLowerCase() === 'content-type') {
+          return;
         }
+        sampleHeaders[header.name] = header.value;
       });
     }
 

src/app/services/actions/samples-action-creators.ts

@@ -45,8 +45,7 @@ export function fetchSamples(): Function {
       const res = await response.json();
       return dispatch(fetchSamplesSuccess(res.sampleQueries));
     } catch (error) {
-      const errorCode = await error.text();
-      return  dispatch(fetchSamplesError({ error: errorCode }));
+      return dispatch(fetchSamplesError({ error }));
     }
   };
 }