allserver 2.4.0 → 2.5.0

package/README.md

@@ -6,8 +6,11 @@ Think of Remote Procedure Calls using exactly the same client and server code bu
 
 Should be used in (micro)services where JavaScript is able to run - your computer, Docker, k8s, virtual machines, serverless functions (Lambdas, Google Cloud Functions, Azure Functions, etc), RaspberryPI, SharedWorker, thread, you name it.
 
+> Uber [moved](https://www.infoq.com/news/2023/10/uber-up-cloud-microservices/) most of its containerized microservices from µDeploy to a new multi-cloud platform named Up in preparation for migrating a considerable portion of its compute footprint to the cloud. **The company spent two years** working on making its many microservices portable to migrate between different computing infrastructures and container management platforms.
+
 Superpowers the `Allserver` gives you:
 
+- Move your code around infrastructures, deployment types, runtimes, platforms, etc with ease.
 - Call gRPC server methods from browser/curl/Postman.
 - Run your HTTP server as gRPC with a single line change (almost).
 - Serve same logic via HTTP and gRPC (or more) simultaneously in the same node.js process.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "allserver",
-  "version": "2.4.0",
+  "version": "2.5.0",
   "description": "Multi-protocol simple RPC server and [optional] client. Boilerplate-less. Opinionated. Minimalistic. DX-first.",
   "main": "src/index.js",
   "scripts": {

package/src/client/AllserverClient.js

@@ -203,8 +203,8 @@ module.exports = require("stampit")({
             this[p].transport = getTransport()({ uri });
         }
 
-        if (before) this[p].before = [].concat(before).concat(this[p].before);
-        if (after) this[p].after = [].concat(after).concat(this[p].after);
+        if (before) this[p].before = [].concat(this[p].before).concat(before).filter(isFunction);
+        if (after) this[p].after = [].concat(this[p].after).concat(after).filter(isFunction);
     },
 
     methods: {
@@ -335,8 +335,9 @@ module.exports = require("stampit")({
             before,
             after,
         } = {}) {
-            if (before) before = [].concat(before);
-            if (after) after = [].concat(after);
+            if (before != null) before = (Array.isArray(before) ? before : [before]).filter(isFunction);
+            if (after != null) after = (Array.isArray(after) ? after : [after]).filter(isFunction);
+
             return this.deepProps({
                 [p]: {
                     transport,

package/src/client/LambdaClientTransport.js

@@ -41,7 +41,7 @@ module.exports = require("./ClientTransport").compose({
             });
             if (typeof promise.promise === "function") promise = promise.promise(); // AWS SDK v2 adoption
             const invocationResponse = await promise;
-            return JSON.parse(invocationResponse.Payload);
+            return JSON.parse(Buffer.from(invocationResponse.Payload));
         },
 
         createCallContext(defaultCtx) {

package/src/server/Allserver.js

@@ -1,17 +1,20 @@
 const assert = require("assert");
 
-const { isObject, isBoolean, isFunction } = require("../util");
+const { isObject, isBoolean, isFunction, uniq } = require("../util");
 
 module.exports = require("stampit")({
     name: "Allserver",
 
+    deepProps: {
+        before: null,
+        after: null,
+    },
+
     props: {
         procedures: {},
         transport: null,
         logger: console,
         introspection: true,
-        before: null,
-        after: null,
 
         callsCount: 0,
     },
@@ -21,8 +24,8 @@ module.exports = require("stampit")({
         this.transport = transport || this.transport || require("./HttpTransport")();
         this.logger = logger || this.logger;
         this.introspection = introspection != null ? introspection : this.introspection;
-        this.before = before || this.before;
-        this.after = after || this.after;
+        if (before) this.before = uniq([].concat(this.before).concat(before).filter(isFunction));
+        if (after) this.after = uniq([].concat(this.after).concat(after).filter(isFunction));
 
         this._validateProcedures();
     },
@@ -178,7 +181,13 @@ module.exports = require("stampit")({
 
     statics: {
         defaults({ procedures, transport, logger, introspection, before, after } = {}) {
-            return this.props({ procedures, transport, logger, introspection, before, after });
+            if (before != null) before = (Array.isArray(before) ? before : [before]).filter(isFunction);
+            if (after != null) after = (Array.isArray(after) ? after : [after]).filter(isFunction);
+
+            return this.compose({
+                props: { procedures, transport, logger, introspection },
+                deepProps: { before, after },
+            });
         },
     },
 });

package/src/util.js

@@ -5,4 +5,5 @@ module.exports = {
     isFunction: (o) => is(o, "function"),
     isObject: (o) => is(o, "object"),
     isPlainObject: (o) => o && o.constructor === Object,
+    uniq: (a) => Array.from(new Set(a)),
 };