Use error handling from apollo-server; throwError should only be defined on server

packages/vulcan-lib/lib/modules/errors.js

@@ -1,4 +1,5 @@
-import { createError } from 'apollo-errors';
+import get from 'lodash/get';
+import isEmpty from 'lodash/isEmpty';
 
 /*
 
@@ -9,8 +10,8 @@ const getFirstWord = input => {
   const parts = /"([^"]*)"/.exec(input);
   if (parts === null) {
     return null;
-   }
-   return parts[1];
+  }
+  return parts[1];
 };
 
 /* 
@@ -27,7 +28,6 @@ In field "addresses": Expected "[JSON]!", found null."
 */
 
 const parseErrorMessage = message => {
-
   // note: optionally add .slice(1) at the end to get rid of the first error, which is not that helpful
   let fieldErrors = message.split('\n');
 
@@ -46,7 +46,7 @@ const parseErrorMessage = message => {
     } else {
       // other generic GraphQL errors
       return {
-        message: error
+        message: error,
       };
     }
   });
@@ -71,19 +71,21 @@ Scenario 4: single GraphQL error with no data property
 */
 export const getErrors = error => {
 
-  // 1. wrap in array
+  // 1. by default, return raw error wrapped in array
   let errors = [error];
+
   // if this is one or more GraphQL errors, extract and convert them
   if (error.graphQLErrors && error.graphQLErrors.length > 0) {
-    // get graphQL error (see https://github.com/thebigredgeek/apollo-errors/issues/12)
+    // get first graphQL error (see https://github.com/thebigredgeek/apollo-errors/issues/12)
     const graphQLError = error.graphQLErrors[0];
-    if (graphQLError.data && !_.isEmpty(graphQLError.data)) {
-      if (graphQLError.data.errors) {
+    const data = get(graphQLError, 'extensions.exception.data')
+    if (data && !isEmpty(data)) {
+      if (data.errors) {
         // 2. there are multiple errors on the data.errors object
-        errors = graphQLError.data && graphQLError.data.errors;
+        errors = data.errors;
       } else {
         // 3. there is only one error
-        errors = [graphQLError.data];
+        errors = [data];
       }
     } else {
       // 4. there is no data object, try to parse raw error message
@@ -92,18 +94,3 @@ export const getErrors = error => {
   }
   return errors;
 };
-
-/*
-
-An error should have: 
-
-- id: will be used as i18n key (note: available as `name` on the client)
-- message: optionally, a plain-text message
-- data: data/values to give more context to the error
-
-*/
-export const throwError = error => {
-  const { id, message = id, data } = error;
-  const MissingDocumentError = createError(id, { message });
-  throw new MissingDocumentError({ id, data });
-};

packages/vulcan-lib/lib/server/errors.js

@@ -0,0 +1,15 @@
+import { ApolloError } from 'apollo-server';
+
+/*
+
+An error should have: 
+
+- id: will be used as i18n key (note: available as `name` on the client)
+- message: optionally, a plain-text message
+- data: data/values to give more context to the error
+
+*/
+export const throwError = error => {
+  const { id, } = error;
+  throw new ApolloError(id, 'VALIDATION_ERROR', error);
+};

packages/vulcan-lib/lib/server/main.js

@@ -8,6 +8,7 @@ export * from './connectors.js';
 export * from './query.js';
 export * from '../modules/index.js';
 export * from './mutators.js';
+export * from './errors.js';
 // TODO: what to do with this?
 export * from './meteor_patch.js';
 //export * from './render_context.js';

packages/vulcan-lib/lib/server/mutators.js

@@ -40,7 +40,7 @@ import {
 } from '../modules/validation.js';
 import { registerSetting } from '../modules/settings.js';
 import { debug, debugGroup, debugGroupEnd } from '../modules/debug.js';
-import { throwError } from '../modules/errors.js';
+import { throwError } from './errors.js';
 import { Connectors } from './connectors.js';
 import pickBy from 'lodash/pickBy';
 import clone from 'lodash/clone';