@orion-js/http 3.0.24 → 3.0.37

package/lib/errors.js

@@ -11,10 +11,12 @@ const defaultOnError = async (req, res, error) => {
         if (error.code === 'AuthError') {
             statusCode = 401;
         }
+        else {
+            console.warn(`[route/handler] OrionError in ${req.path}:`, error);
+        }
         const data = error.getInfo();
-        res.writeHead(statusCode);
-        res.end(JSON.stringify(data, null, 2));
-        console.warn(`[route/handler] OrionError in ${req.path}:`, error);
+        res.status(statusCode);
+        res.json(data);
     }
     else if (error.isGraphQLError) {
         res.writeHead(error.statusCode);

package/lib/viewer.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.setGetViewer = exports.getViewer = void 0;
+const helpers_1 = require("@orion-js/helpers");
 global.getViewerRef = () => null;
 const getViewer = async (req) => {
     try {
@@ -10,8 +11,7 @@ const getViewer = async (req) => {
         return viewer;
     }
     catch (err) {
-        console.error('Orion HTTP Error: error getting viewer: ', err);
-        return {};
+        throw new helpers_1.UserError('AuthError', err.message);
     }
 };
 exports.getViewer = getViewer;

package/lib/viewer.test.js

@@ -9,7 +9,7 @@ const route_1 = __importDefault(require("./routes/route"));
 const registerRoute_1 = __importDefault(require("./routes/registerRoute"));
 const supertest_1 = __importDefault(require("supertest"));
 describe('Test viewer', () => {
-    test('Is should pass the correct viewer', async () => {
+    test('It should pass the correct viewer', async () => {
         (0, viewer_1.setGetViewer)(async (req) => {
             return {
                 name: req.params.name,
@@ -29,4 +29,23 @@ describe('Test viewer', () => {
         const response = await (0, supertest_1.default)(app).post('/testViewer/nico').send({ lastName: 'López' });
         expect(response.body).toEqual({ name: 'nico', lastName: 'López' });
     });
+    test('It should throw an error correctly when getViewer returns an error', async () => {
+        (0, viewer_1.setGetViewer)(async (req) => {
+            throw new Error('invalid headers');
+        });
+        const testRoute = (0, route_1.default)({
+            path: '/testViewer/:name',
+            method: 'post',
+            bodyParser: 'json',
+            async resolve(req, res, viewer) {
+                return { body: viewer };
+            }
+        });
+        (0, registerRoute_1.default)(testRoute);
+        const app = (0, start_1.getApp)();
+        const response = await (0, supertest_1.default)(app).post('/testViewer/nico').send({ lastName: 'López' });
+        // expect reponse code to be 401
+        expect(response.status).toBe(401);
+        expect(response.body).toEqual({ error: 'AuthError', message: 'invalid headers' });
+    });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@orion-js/http",
-  "version": "3.0.24",
+  "version": "3.0.37",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
   "files": [
@@ -17,9 +17,9 @@
     "upgrade-interactive": "yarn upgrade-interactive"
   },
   "dependencies": {
-    "@orion-js/helpers": "^3.0.17",
-    "@orion-js/resolvers": "^3.0.24",
-    "@orion-js/schema": "^3.0.17",
+    "@orion-js/helpers": "^3.0.36",
+    "@orion-js/resolvers": "^3.0.37",
+    "@orion-js/schema": "^3.0.37",
     "body-parser": "1.19.0",
     "express": "4.17.1"
   },
@@ -39,5 +39,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "5ac1c68e72af3dbeaf8eb5c2a04d5325da2a7e3f"
+  "gitHead": "741691d7603f3d42e6e3e8fa9dc0f9de51a29424"
 }