@fastify/reply-from 11.0.1 → 11.0.2

package/.github/workflows/ci.yml

@@ -17,7 +17,7 @@ on:
 
 jobs:
   test:
-    uses: fastify/workflows/.github/workflows/plugins-ci.yml@v5.0.0
+    uses: fastify/workflows/.github/workflows/plugins-ci.yml@v5
     with:
       license-check: true
       lint: true

package/index.js

@@ -20,7 +20,8 @@ const {
   ConnectionResetError,
   ConnectTimeoutError,
   UndiciSocketError,
-  InternalServerError
+  InternalServerError,
+  BadGatewayError
 } = require('./lib/errors')
 
 const fastifyReplyFrom = fp(function from (fastify, opts, next) {
@@ -199,7 +200,13 @@ const fastifyReplyFrom = fp(function from (fastify, opts, next) {
       } else {
         copyHeaders(rewriteHeaders(res.headers, this.request), this)
       }
-      this.code(res.statusCode)
+      try {
+        this.code(res.statusCode)
+      } catch (err) {
+        // Since we know `FST_ERR_BAD_STATUS_CODE` will be recieved
+        onError(this, { error: new BadGatewayError() })
+        this.request.log.warn(err, 'response has invalid status code')
+      }
       if (onResponse) {
         onResponse(this.request, this, res)
       } else {

package/lib/errors.js

@@ -12,3 +12,4 @@ module.exports.ConnectionResetError = createError('ECONNRESET', 'Connection Rese
 module.exports.ConnectTimeoutError = createError('UND_ERR_CONNECT_TIMEOUT', 'Connect Timeout Error', 500)
 module.exports.UndiciSocketError = createError('UND_ERR_SOCKET', 'Undici Socket Error', 500)
 module.exports.InternalServerError = createError('FST_REPLY_FROM_INTERNAL_SERVER_ERROR', '%s', 500)
+module.exports.BadGatewayError = createError('FST_REPLY_FROM_BAD_GATEWAY', 'Bad Gateway', 502)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fastify/reply-from",
-  "version": "11.0.1",
+  "version": "11.0.2",
   "description": "forward your HTTP request to another server, for fastify",
   "main": "index.js",
   "type": "commonjs",
@@ -56,7 +56,7 @@
     "end-of-stream": "^1.4.4",
     "fast-content-type-parse": "^2.0.0",
     "fast-querystring": "^1.1.2",
-    "fastify-plugin": "^4.5.1",
+    "fastify-plugin": "^5.0.1",
     "toad-cache": "^3.7.0",
     "undici": "^6.11.1"
   },

package/test/on-invalid-upstream-response.test.js

@@ -0,0 +1,52 @@
+'use strict'
+
+const t = require('tap')
+const Fastify = require('fastify')
+const From = require('..')
+const http = require('node:http')
+const get = require('simple-get').concat
+
+const instance = Fastify()
+instance.register(From)
+
+t.plan(8)
+t.teardown(instance.close.bind(instance))
+
+const target = http.createServer((req, res) => {
+  t.pass('request proxied')
+  t.equal(req.method, 'GET')
+  res.statusCode = 888
+  res.end('non-standard status code')
+})
+
+instance.get('/', (_, reply) => {
+  reply.from(`http://localhost:${target.address().port}`, {
+    onResponse: (_, reply, res) => {
+      t.equal(res.statusCode, 888)
+    }
+  })
+})
+
+t.teardown(target.close.bind(target))
+
+instance.listen({ port: 0 }, (err) => {
+  t.error(err)
+
+  target.listen({ port: 0 }, (err) => {
+    t.error(err)
+
+    get(
+      `http://localhost:${instance.server.address().port}`,
+      (err, res, data) => {
+        t.error(err)
+        t.equal(res.statusCode, 502)
+        t.same(JSON.parse(data), {
+          statusCode: 502,
+          code: 'FST_REPLY_FROM_BAD_GATEWAY',
+          error: 'Bad Gateway',
+          message: 'Bad Gateway'
+        })
+      }
+    )
+  })
+})