Improve error handling for circular error objs

Author: larixer

src/common/log.js

@@ -10,9 +10,15 @@ if (__DEV__ && __SERVER__) {
     if (arguments.length == 1 && typeof arguments[0] === 'string' && arguments[0].match(/^\[(HMR|WDS)\]/)) {
       console_log('backend ' + arguments[0]);
     } else {
-      console_log.apply(console_log, arguments);
+      console_log.apply(global.console, arguments);
     }
   };
+
+  // let console_err = global.console.error;
+  // global.console.error = function() {
+  //   arguments[0] = 'ce ' + new Error().stack + '\n\n\n' + arguments[0];
+  //   console_err.apply(global.console, arguments);
+  // };
 }
 
 export default log;

src/server/api/schema.js

@@ -1,15 +1,12 @@
-import { makeExecutableSchema, addErrorLoggingToSchema } from 'graphql-tools';
+import { makeExecutableSchema } from 'graphql-tools';
 
 import rootSchemaDef from './rootSchema.graphqls';
 import modules from '../modules';
 import pubsub from './pubsub';
-import log from '../../common/log';
 
 const executableSchema = makeExecutableSchema({
   typeDefs: [rootSchemaDef].concat(modules.schemas),
   resolvers: modules.createResolvers(pubsub)
 });
 
-addErrorLoggingToSchema(executableSchema, { log: e => log.error(e) });
-
 export default executableSchema;

src/server/middleware/error.js

@@ -1,10 +1,25 @@
 import path from 'path';
 import fs from 'fs';
+import serialize from 'serialize-javascript';
 import log from '../../common/log';
 import { options as spinConfig } from '../../../.spinrc.json';
 
 let assetMap;
 
+const stripCircular = (from, seen) => {
+  const to = Array.isArray(from) ? [] : {};
+  seen = seen || [];
+  seen.push(from);
+  Object.getOwnPropertyNames(from).forEach(key => {
+    if (!from[key] || (typeof from[key] !== 'object' && !Array.isArray(from[key]))) {
+      to[key] = from[key];
+    } else if (seen.indexOf(from[key]) < 0) {
+      to[key] = stripCircular(from[key], seen.slice(0));
+    } else to[key] = '[Circular]';
+  });
+  return to;
+};
+
 /* 
  * The code below MUST be declared as a function, not closure, 
  * otherwise Express will fail to execute this handler
@@ -16,10 +31,7 @@ function errorMiddleware(e, req, res, next) {
     assetMap = JSON.parse(fs.readFileSync(path.join(spinConfig.frontendBuildDir, 'web', 'assets.json')));
   }
 
-  const serverErrorScript = `<script charset="UTF-8">window.__SERVER_ERROR__=${JSON.stringify(
-    e,
-    Object.getOwnPropertyNames(e)
-  )};</script>`;
+  const serverErrorScript = `<script charset="UTF-8">window.__SERVER_ERROR__=${serialize(stripCircular(e))};</script>`;
   const vendorScript = assetMap['vendor.js'] ? `<script src="/${assetMap['vendor.js']}" charSet="utf-8"></script>` : '';
 
   res.status(200).send(

src/server/middleware/graphql.js

@@ -10,8 +10,9 @@ export default graphqlExpress(async (req, res, next) => {
     return {
       schema,
       context: await modules.createContext(req, res),
-      tracing: !!settings.analytics.apolloEngine.key,
-      cacheControl: !!settings.analytics.apolloEngine.key
+      debug: false,
+      tracing: !!settings.engine.engineConfig.apiKey,
+      cacheControl: !!settings.engine.engineConfig.apiKey
     };
   } catch (e) {
     next(e);

src/server/middleware/website.jsx

@@ -30,19 +30,16 @@ const { protocol, hostname, port, pathname } = url.parse(__BACKEND_URL__);
 const apiUrl = `${protocol}//${hostname}:${process.env.PORT || port}${pathname}`;
 
 const renderServerSide = async (req, res) => {
+  console.log('req.path:', req.path);
   // if (__PERSIST_GQL__) {
   //   networkInterface = addPersistedQueries(networkInterface, queryMap);
   // }
   //
 
   const fetch = createApolloFetch({ uri: apiUrl, constructOptions: modules.constructFetchOptions });
   fetch.batchUse(({ options }, next) => {
-    try {
-      options.credentials = 'include';
-      options.headers = req.headers;
-    } catch (e) {
-      console.error(e);
-    }
+    options.credentials = 'include';
+    options.headers = req.headers;
 
     next();
   });