@platformatic/composer 2.71.0-alpha.1 → 2.71.0

package/lib/proxy.js

@@ -45,7 +45,7 @@ async function resolveServiceProxyParameters (service) {
     prefix,
     rewritePrefix,
     internalRewriteLocationHeader,
-    needsRootRedirect: meta.needsRootRedirect,
+    needsRootTrailingSlash: meta.needsRootTrailingSlash,
     needsRefererBasedRedirect: meta.needsRefererBasedRedirect,
     upstream: service.proxy?.upstream,
     ws: service.proxy?.ws
@@ -72,7 +72,7 @@ module.exports = fp(async function (app, opts) {
       url,
       rewritePrefix,
       internalRewriteLocationHeader,
-      needsRootRedirect,
+      needsRootTrailingSlash,
       needsRefererBasedRedirect,
       ws
     } = parameters
@@ -81,34 +81,16 @@ module.exports = fp(async function (app, opts) {
     const basePath = `/${prefix ?? ''}`.replaceAll(/\/+/g, '/').replace(/\/$/, '')
     const dispatcher = getGlobalDispatcher()
 
-    if (needsRootRedirect) {
-      app.addHook('preHandler', (req, reply, done) => {
-        if (req.url === basePath) {
-          app.inject(
-            {
-              method: req.method,
-              url: `${basePath}/`,
-              headers: req.headers,
-              payload: req.body
-            },
-            (err, result) => {
-              if (err) {
-                done(err)
-                return
-              }
-
-              const replyHeaders = result.headers
-              delete replyHeaders['content-length']
-              delete replyHeaders['transfer-encoding']
-
-              reply.code(result.statusCode).headers(replyHeaders).send(result.rawPayload)
-              done()
-            }
-          )
-        } else {
-          done()
+    let preRewrite = null
+
+    if (needsRootTrailingSlash) {
+      preRewrite = function preRewrite (url) {
+        if (url === basePath) {
+          url += '/'
         }
-      })
+
+        return url
+      }
     }
 
     /*
@@ -118,7 +100,7 @@ module.exports = fp(async function (app, opts) {
       from the Referer header.
     */
     if (needsRefererBasedRedirect) {
-      app.addHook('preHandler', (req, reply, done) => {
+      app.addHook('preHandler', function refererBasedRedirectPreHandler (req, reply, done) {
         // If the URL is already targeted to the service, do nothing
         if (req.url.startsWith(basePath)) {
           done()
@@ -164,11 +146,12 @@ module.exports = fp(async function (app, opts) {
       : null
 
     const proxyOptions = {
-      websocket: true,
       prefix,
       rewritePrefix,
       upstream: service.proxy?.upstream ?? origin,
+      preRewrite,
 
+      websocket: true,
       wsUpstream: ws?.upstream ?? url ?? origin,
       wsReconnect: ws?.reconnect,
       wsHooks: {
@@ -185,6 +168,7 @@ module.exports = fp(async function (app, opts) {
       config: {
         [kProxyRoute]: true
       },
+
       internalRewriteLocationHeader: false,
       replyOptions: {
         rewriteHeaders: headers => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@platformatic/composer",
-  "version": "2.71.0-alpha.1",
+  "version": "2.71.0",
   "description": "",
   "main": "index.js",
   "bin": {
@@ -32,20 +32,20 @@
     "typescript": "^5.5.4",
     "why-is-node-running": "2",
     "ws": "^8.16.0",
-    "@platformatic/client": "2.71.0-alpha.1",
-    "@platformatic/config": "2.71.0-alpha.1",
-    "@platformatic/db": "2.71.0-alpha.1"
+    "@platformatic/config": "2.71.0",
+    "@platformatic/client": "2.71.0",
+    "@platformatic/db": "2.71.0"
   },
   "dependencies": {
     "@fastify/error": "^4.0.0",
-    "@fastify/http-proxy": "^11.1.1",
+    "@fastify/http-proxy": "^11.3.0",
     "@fastify/reply-from": "^12.0.0",
     "@fastify/static": "^8.0.0",
     "@fastify/swagger": "^9.0.0",
     "@fastify/view": "^10.0.1",
     "@platformatic/fastify-openapi-glue": "^5.1.0",
     "@platformatic/graphql-composer": "^0.10.0",
-    "@scalar/fastify-api-reference": "1.31.8",
+    "@scalar/fastify-api-reference": "1.31.14",
     "ajv": "^8.12.0",
     "commist": "^3.2.0",
     "console-table-printer": "^2.12.0",
@@ -68,12 +68,12 @@
     "rfdc": "^1.3.1",
     "semgrator": "^0.3.0",
     "undici": "^7.0.0",
-    "@platformatic/config": "2.71.0-alpha.1",
-    "@platformatic/generators": "2.71.0-alpha.1",
-    "@platformatic/scalar-theme": "2.71.0-alpha.1",
-    "@platformatic/telemetry": "2.71.0-alpha.1",
-    "@platformatic/utils": "^2.71.0-alpha.1",
-    "@platformatic/service": "2.71.0-alpha.1"
+    "@platformatic/config": "2.71.0",
+    "@platformatic/scalar-theme": "2.71.0",
+    "@platformatic/generators": "2.71.0",
+    "@platformatic/service": "2.71.0",
+    "@platformatic/telemetry": "2.71.0",
+    "@platformatic/utils": "^2.71.0"
   },
   "scripts": {
     "test": "pnpm run lint && borp -T --timeout=1200000 -c 1 && tsd",

package/schema.json

@@ -1,5 +1,5 @@
 {
-  "$id": "https://schemas.platformatic.dev/@platformatic/composer/2.71.0-alpha.1.json",
+  "$id": "https://schemas.platformatic.dev/@platformatic/composer/2.71.0.json",
   "$schema": "http://json-schema.org/draft-07/schema#",
   "title": "Platformatic Composer",
   "type": "object",