@fastify/reply-from 9.0.0 → 9.0.1

package/.taprc

@@ -8,4 +8,4 @@ statements: 96
 branches: 96
 
 files:
-  - test/**/*.js
+  - test/**/*.test.js

package/lib/request.js

@@ -112,6 +112,10 @@ function buildRequest (opts) {
     })
     req.on('error', done)
     req.on('response', res => {
+      // remove timeout for sse connections
+      if (res.headers['content-type'] === 'text/event-stream') {
+        req.setTimeout(0)
+      }
       done(null, { statusCode: res.statusCode, headers: res.headers, stream: res })
     })
     req.once('timeout', () => {
@@ -210,6 +214,12 @@ function buildRequest (opts) {
       if (err) done(err)
     })
     req.on('response', headers => {
+      // remove timeout for sse connections
+      if (headers['content-type'] === 'text/event-stream') {
+        req.setTimeout(0)
+        http2Client.setTimeout(0)
+      }
+
       const statusCode = headers[':status']
       done(null, { statusCode, headers, stream: req })
     })
@@ -250,7 +260,7 @@ function getHttp2Opts (opts) {
   http2Opts.sessionOptions = http2Opts.sessionOptions || {}
 
   if (!http2Opts.sessionTimeout) {
-    http2Opts.sessionTimeout = opts.sessionTimeout || 6000
+    http2Opts.sessionTimeout = opts.sessionTimeout || 60000
   }
   if (!http2Opts.requestTimeout) {
     http2Opts.requestTimeout = 10000

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fastify/reply-from",
-  "version": "9.0.0",
+  "version": "9.0.1",
   "description": "forward your HTTP request to another server, for fastify",
   "main": "index.js",
   "types": "types/index.d.ts",
@@ -46,7 +46,7 @@
     "split2": "^4.1.0",
     "standard": "^17.0.0",
     "tap": "^16.2.0",
-    "tsd": "^0.25.0"
+    "tsd": "^0.26.0"
   },
   "dependencies": {
     "@fastify/error": "^3.0.0",

package/test/http-timeout.test.js

@@ -0,0 +1,82 @@
+'use strict'
+
+const { test } = require('tap')
+const Fastify = require('fastify')
+const From = require('..')
+const got = require('got')
+const FakeTimers = require('@sinonjs/fake-timers')
+
+const clock = FakeTimers.createClock()
+
+test('http request timeout', async (t) => {
+  t.autoend(false)
+
+  const target = Fastify()
+  t.teardown(target.close.bind(target))
+
+  target.get('/', (request, reply) => {
+    t.pass('request arrives')
+
+    clock.setTimeout(() => {
+      reply.status(200).send('hello world')
+      t.end()
+    }, 200)
+  })
+
+  await target.listen({ port: 0 })
+
+  const instance = Fastify()
+  t.teardown(instance.close.bind(instance))
+
+  instance.register(From, { http: { requestOptions: { timeout: 100 } } })
+
+  instance.get('/', (request, reply) => {
+    reply.from(`http://localhost:${target.server.address().port}/`)
+  })
+
+  await instance.listen({ port: 0 })
+
+  try {
+    await got.get(`http://localhost:${instance.server.address().port}/`, { retry: 0 })
+  } catch (err) {
+    t.equal(err.response.statusCode, 504)
+    t.match(err.response.headers['content-type'], /application\/json/)
+    t.same(JSON.parse(err.response.body), {
+      statusCode: 504,
+      code: 'FST_REPLY_FROM_GATEWAY_TIMEOUT',
+      error: 'Gateway Timeout',
+      message: 'Gateway Timeout'
+    })
+    clock.tick(200)
+    return
+  }
+
+  t.fail()
+})
+
+test('http sse removes timeout test', async (t) => {
+  const target = Fastify()
+  t.teardown(target.close.bind(target))
+
+  target.get('/', (request, reply) => {
+    t.pass('request arrives')
+
+    reply.header('content-type', 'text/event-stream').status(200).send('hello world')
+  })
+
+  await target.listen({ port: 0 })
+
+  const instance = Fastify()
+  t.teardown(instance.close.bind(instance))
+
+  instance.register(From, { http: { requestOptions: { timeout: 100 } } })
+
+  instance.get('/', (request, reply) => {
+    reply.from(`http://localhost:${target.server.address().port}/`)
+  })
+
+  await instance.listen({ port: 0 })
+
+  const { statusCode } = await got.get(`http://localhost:${instance.server.address().port}/`, { retry: 0 })
+  t.equal(statusCode, 200)
+})

package/test/{http2-target-multi-crash.js → http2-target-multi-crash.test.js}

@@ -16,7 +16,9 @@ test('http -> http2 crash multiple times', async (t) => {
 
   instance.register(From, {
     base: 'http://localhost:3128',
-    http2: true,
+    http2: {
+      sessionTimeout: 6000
+    },
     sessionTimeout: 200
   })
 

package/test/{http2-timeout.js → http2-timeout.test.js}

@@ -20,7 +20,7 @@ test('http2 request timeout', async (t) => {
 
   instance.register(From, {
     base: `http://localhost:${target.server.address().port}`,
-    http2: { requestTimeout: 100 }
+    http2: { requestTimeout: 100, sessionTimeout: 6000 }
   })
 
   instance.get('/', (request, reply) => {
@@ -92,3 +92,34 @@ test('http2 session timeout', async (t) => {
 
   t.fail()
 })
+
+test('http2 sse removes request and session timeout test', async (t) => {
+  const target = Fastify({ http2: true, sessionTimeout: 0 })
+
+  target.get('/', (request, reply) => {
+    t.pass('request arrives')
+
+    reply.status(200).header('content-type', 'text/event-stream').send('hello world')
+  })
+
+  await target.listen({ port: 0 })
+
+  const instance = Fastify()
+
+  instance.register(From, {
+    base: `http://localhost:${target.server.address().port}`,
+    http2: { sessionTimeout: 100 }
+  })
+
+  instance.get('/', (request, reply) => {
+    reply.from(`http://localhost:${target.server.address().port}/`)
+  })
+
+  await instance.listen({ port: 0 })
+
+  t.teardown(instance.close.bind(instance))
+  t.teardown(target.close.bind(target))
+
+  const { statusCode } = await got.get(`http://localhost:${instance.server.address().port}/`, { retry: 0 })
+  t.equal(statusCode, 200)
+})

package/test/http-timeout.js

@@ -1,57 +0,0 @@
-'use strict'
-
-const t = require('tap')
-const Fastify = require('fastify')
-const From = require('..')
-const got = require('got')
-const FakeTimers = require('@sinonjs/fake-timers')
-
-const clock = FakeTimers.createClock()
-
-t.autoend(false)
-
-const target = Fastify()
-t.teardown(target.close.bind(target))
-
-target.get('/', (request, reply) => {
-  t.pass('request arrives')
-
-  clock.setTimeout(() => {
-    reply.status(200).send('hello world')
-    t.end()
-  }, 200)
-})
-
-async function main () {
-  await target.listen({ port: 0 })
-
-  const instance = Fastify()
-  t.teardown(instance.close.bind(instance))
-
-  instance.register(From, { http: { requestOptions: { timeout: 100 } } })
-
-  instance.get('/', (request, reply) => {
-    reply.from(`http://localhost:${target.server.address().port}/`)
-  })
-
-  await instance.listen({ port: 0 })
-
-  try {
-    await got.get(`http://localhost:${instance.server.address().port}/`, { retry: 0 })
-  } catch (err) {
-    t.equal(err.response.statusCode, 504)
-    t.match(err.response.headers['content-type'], /application\/json/)
-    t.same(JSON.parse(err.response.body), {
-      statusCode: 504,
-      code: 'FST_REPLY_FROM_GATEWAY_TIMEOUT',
-      error: 'Gateway Timeout',
-      message: 'Gateway Timeout'
-    })
-    clock.tick(200)
-    return
-  }
-
-  t.fail()
-}
-
-main()