unnbound-events 2.1.0-beta.1 → 2.1.0-beta.2

package/dist/lib/routing/as2.js

@@ -5,6 +5,32 @@ const unnbound_logger_sdk_1 = require("unnbound-logger-sdk");
 const error_1 = require("../error");
 const utils_1 = require("../utils");
 const endpoint_1 = require("./endpoint");
+/**
+ * Transient platform failures a partner should re-send against. `As2SdkError` already computed
+ * these from the platform response, but it arrives here as a plain unknown — this package cannot
+ * import `@ontemper/as2` (the dependency runs the other way), so the shape is read defensively.
+ */
+const RETRYABLE_AS2_STATUSES = [502, 503, 504];
+/**
+ * AS2 partners drive their re-send behaviour off the status code, so collapsing every failure to
+ * a flat 500 throws away the one signal telling them to try again — the same signal the
+ * in-flight/parked path already sends deliberately as a 503. Lift the transient statuses onto the
+ * wire; the message stays generic so no internal detail reaches the partner.
+ */
+const toWireFailure = (error) => {
+    if (error && typeof error === 'object') {
+        const { name, status } = error;
+        const retryable = RETRYABLE_AS2_STATUSES.find((candidate) => candidate === status);
+        if (name === 'As2SdkError' && retryable) {
+            return new error_1.ServerError({
+                message: 'AS2 message could not be processed; retry later',
+                status: retryable,
+                cause: error,
+            });
+        }
+    }
+    return error_1.ServerError.fromError(error);
+};
 /**
  * Verify-then-bind AS2 endpoint. The public `x-unnbound-trace-id` header is never AS2
  * continuation authority: the route starts on a fresh trace, runs `prepare` (the hidden
@@ -38,7 +64,7 @@ const as2Endpoint = (prepare, handler) => {
         }
         catch (error) {
             return (0, unnbound_logger_sdk_1.withTrace)(async () => {
-                const failure = error_1.ServerError.fromError(error);
+                const failure = toWireFailure(error);
                 // Bookend the request on its own failure trace: the span records the sanitized
                 // AS2 payload and rethrows; the wire response carries no protocol claims.
                 await (0, unnbound_logger_sdk_1.startSpan)('Event request', () => Promise.reject(failure), (o) => (0, utils_1.buildAs2IncomingPayload)(event, o)).catch(() => undefined);
@@ -61,7 +87,9 @@ const as2Endpoint = (prepare, handler) => {
             catch (error) {
                 // The span above already logged the failure on the bound trace; respond with the
                 // sanitized shape instead of rethrowing into the header-derived middleware trace.
-                const result = error_1.ServerError.fromError(error).toJson();
+                // `acknowledge()` runs in here too, so a transient platform failure keeps its
+                // retryable status rather than becoming an indistinguishable 500.
+                const result = toWireFailure(error).toJson();
                 return { ...result, headers: (0, endpoint_1.attachTraceHeaders)(result.headers) };
             }
         }, { traceId });

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "unnbound-events",
   "description": "Unified events SDK to handle HTTP routes and queued messages with a single routing API.",
-  "version": "2.1.0-beta.1",
+  "version": "2.1.0-beta.2",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "author": "Unnbound Team",