@wix/be-server - npm Package Compare versions

Comparing version 1.0.0 to 1.0.1

lib/routes.js

@@ -65,3 +65,3 @@ const url = require('url');
       return {
-        ...resolvedRoute,
+        route: resolvedRoute,
         request
@@ -68,0 +68,0 @@ };

lib/routes.spec.js

@@ -23,5 +23,5 @@ const routes = require('./routes');
 
-      const {request, method} = givenRoutes.resolve('GET', '/api/path-1?valueFromQuery=query-1');
+      const {request, route} = givenRoutes.resolve('GET', '/api/path-1?valueFromQuery=query-1');
 
-      expect(method.name).to.equal('Get');
+      expect(route.method.name).to.equal('Get');
       expect(request).to.deep.equal({
@@ -46,5 +46,5 @@ valueFromPath: 'path-1',
 
-    const {request, method} = givenRoutes.resolve('GET', '/api/path-1?test.arr=arr-1&test.arr=arr-2&test.bool=true&num=10');
+    const {request, route} = givenRoutes.resolve('GET', '/api/path-1?test.arr=arr-1&test.arr=arr-2&test.bool=true&num=10');
 
-    expect(method.name).to.equal('Get');
+    expect(route.method.name).to.equal('Get');
     expect(request).to.deep.equal({
@@ -51,0 +51,0 @@ test: {

lib/server.js

@@ -9,6 +9,4 @@ const http = require('http');
     try {
-      const httpMethod = req.method;
+      const {route, request} = methodRoutes.resolve(req.method, req.url);
 
-      const route = methodRoutes.resolve(httpMethod, req.url);
-
       if (!route) {
@@ -18,8 +16,20 @@ res.statusCode = 404;
 
-      const result = await route.implementation({message: 'Hello'});
+      const contentType = req.headers['content-type'];
 
-      res.write(JSON.stringify(result));
+      if (contentType) {
+        await new Promise((resolve, reject) =>
+          req.on('data', async(data) => {
+            try {
+              resolve(await execute(route, res, JSON.parse(data)));
+            } catch(e) {
+              reject(e);
+            }
+          }));
+      } else {
+        await execute(route, res, request);
+      }
     } catch(e) {
       res.statusCode = 500;
       res.write(e.toString());
+      console.error(e);
     } finally {
@@ -42,1 +52,8 @@ res.end();
 };
+
+async function execute(route, res, body) {
+  const result = await route.implementation(body);
+
+  res.setHeader('Content-Type', 'application/json');
+  res.write(JSON.stringify(result));
+}

package.json

 {
   "name": "@wix/be-server",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "author": "Mantas Indrašius <[e-mail]>",
@@ -29,2 +29,2 @@ "publishConfig": {
   }
-}
+}

test/http-server.e2e.js

@@ -17,3 +17,4 @@ const beServer = require('..');
       .withService('test.EchoService', {
-        echo: (message) => message
+        echo: (message) => message,
+        postEcho: (message) => message
       })
@@ -36,2 +37,20 @@ .start({ port: 9901 });
   });
+
+  it('should post', async() => {
+    const response = await fetch('http://localhost:9901/api/echo', {
+      method: 'POST',
+      headers: {
+        'Content-Type': 'application/json'
+      },
+      body: JSON.stringify({ message: 'Hello' })
+    });
+
+    expect(response.status).to.equal(200);
+
+    const body = await response.json();
+
+    expect(body).to.deep.equal({
+      message: 'Hello'
+    });
+  });
 });

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

10741

increased by11.02%

Lines of code

294

increased by11.36%

Worsened metrics

Number of medium supply chain risk alerts

3

increased by50%

No dependency changes