fastify 5.1.0 → 5.2.1

package/test/web-api.test.js

@@ -227,6 +227,50 @@ test('Error when Response.bodyUsed', async (t) => {
   t.assert.strictEqual(body.code, 'FST_ERR_REP_RESPONSE_BODY_CONSUMED')
 })
 
+test('Error when Response.body.locked', async (t) => {
+  t.plan(3)
+
+  const fastify = Fastify()
+
+  fastify.get('/', async function (request, reply) {
+    const stream = Readable.toWeb(fs.createReadStream(__filename))
+    const response = new Response(stream, {
+      status: 200,
+      headers: {
+        hello: 'world'
+      }
+    })
+    stream.getReader()
+    t.assert.strictEqual(stream.locked, true)
+    return reply.send(response)
+  })
+
+  const response = await fastify.inject({ method: 'GET', path: '/' })
+
+  t.assert.strictEqual(response.statusCode, 500)
+  const body = response.json()
+  t.assert.strictEqual(body.code, 'FST_ERR_REP_READABLE_STREAM_LOCKED')
+})
+
+test('Error when ReadableStream.locked', async (t) => {
+  t.plan(3)
+
+  const fastify = Fastify()
+
+  fastify.get('/', async function (request, reply) {
+    const stream = Readable.toWeb(fs.createReadStream(__filename))
+    stream.getReader()
+    t.assert.strictEqual(stream.locked, true)
+    return reply.send(stream)
+  })
+
+  const response = await fastify.inject({ method: 'GET', path: '/' })
+
+  t.assert.strictEqual(response.statusCode, 500)
+  const body = response.json()
+  t.assert.strictEqual(body.code, 'FST_ERR_REP_READABLE_STREAM_LOCKED')
+})
+
 test('allow to pipe with fetch', async (t) => {
   t.plan(2)
   const abortController = new AbortController()

package/types/hooks.d.ts

@@ -1,4 +1,4 @@
-import { Readable } from 'stream'
+import { Readable } from 'node:stream'
 import { FastifyInstance } from './instance'
 import { RouteOptions, RouteGenericInterface } from './route'
 import { RawServerBase, RawServerDefault, RawRequestDefaultExpression, RawReplyDefaultExpression, ContextConfigDefault } from './utils'

package/types/instance.d.ts

@@ -1,8 +1,8 @@
 import { FastifyError } from '@fastify/error'
 import { ConstraintStrategy, FindResult, HTTPVersion } from 'find-my-way'
-import * as http from 'http'
+import * as http from 'node:http'
 import { InjectOptions, CallbackFunc as LightMyRequestCallback, Chain as LightMyRequestChain, Response as LightMyRequestResponse } from 'light-my-request'
-import { AddressInfo } from 'net'
+import { AddressInfo } from 'node:net'
 import { AddContentTypeParser, ConstructorAction, FastifyBodyParser, ProtoAction, getDefaultJsonParser, hasContentTypeParser, removeAllContentTypeParsers, removeContentTypeParser } from './content-type-parser'
 import { ApplicationHook, HookAsyncLookup, HookLookup, LifecycleHook, onCloseAsyncHookHandler, onCloseHookHandler, onErrorAsyncHookHandler, onErrorHookHandler, onListenAsyncHookHandler, onListenHookHandler, onReadyAsyncHookHandler, onReadyHookHandler, onRegisterHookHandler, onRequestAbortAsyncHookHandler, onRequestAbortHookHandler, onRequestAsyncHookHandler, onRequestHookHandler, onResponseAsyncHookHandler, onResponseHookHandler, onRouteHookHandler, onSendAsyncHookHandler, onSendHookHandler, onTimeoutAsyncHookHandler, onTimeoutHookHandler, preCloseAsyncHookHandler, preCloseHookHandler, preHandlerAsyncHookHandler, preHandlerHookHandler, preParsingAsyncHookHandler, preParsingHookHandler, preSerializationAsyncHookHandler, preSerializationHookHandler, preValidationAsyncHookHandler, preValidationHookHandler } from './hooks'
 import { FastifyBaseLogger, FastifyChildLoggerFactory } from './logger'
@@ -548,6 +548,13 @@ export interface FastifyInstance<
    * Remove all content type parsers, including the default ones
    */
   removeAllContentTypeParsers: removeAllContentTypeParsers
+  /**
+   * Add a non-standard HTTP method
+   *
+   * Methods defined by default include `GET`, `HEAD`, `TRACE`, `DELETE`,
+   * `OPTIONS`, `PATCH`, `PUT` and `POST`
+   */
+  addHttpMethod(method: string, methodOptions?: { hasBody: boolean }): FastifyInstance<RawServer, RawRequest, RawReply, Logger, TypeProvider>;
   /**
    * Fastify default JSON parser
    */

package/types/reply.d.ts

@@ -1,4 +1,4 @@
-import { Buffer } from 'buffer'
+import { Buffer } from 'node:buffer'
 import { FastifyInstance } from './instance'
 import { FastifyBaseLogger } from './logger'
 import { FastifyRequest, RequestRouteOptions } from './request'

package/types/request.d.ts

@@ -87,8 +87,4 @@ export interface FastifyRequest<RouteGeneric extends RouteGenericInterface = Rou
   compileValidationSchema(schema: { [key: string]: any }, httpPart?: HTTPRequestPart): ValidationFunction
   validateInput(input: any, schema: { [key: string]: any }, httpPart?: HTTPRequestPart): boolean
   validateInput(input: any, httpPart?: HTTPRequestPart): boolean
-
-  // Prefer `socket` over deprecated `connection` property in node 13.0.0 or higher
-  // @deprecated
-  readonly connection: RawRequest['socket'];
 }

package/types/serverFactory.d.ts

@@ -1,7 +1,7 @@
 import { RawServerBase, RawServerDefault, RawReplyDefaultExpression, RawRequestDefaultExpression } from './utils'
-import * as http from 'http'
-import * as https from 'https'
-import * as http2 from 'http2'
+import * as http from 'node:http'
+import * as https from 'node:https'
+import * as http2 from 'node:http2'
 
 export type FastifyServerFactoryHandler<
   RawServer extends RawServerBase = RawServerDefault,

package/types/utils.d.ts

@@ -1,6 +1,6 @@
-import * as http from 'http'
-import * as http2 from 'http2'
-import * as https from 'https'
+import * as http from 'node:http'
+import * as http2 from 'node:http2'
+import * as https from 'node:https'
 
 type AutocompletePrimitiveBaseType<T> =
   T extends string ? string :

package/.vscode/settings.json

@@ -1,22 +0,0 @@
-{
-  "workbench.colorCustomizations": {
-    "[GitHub Dark]": {
-      "tab.activeBackground": "#0d0d0d",
-      "tab.activeBorder": "#ffff00"
-    },
-    "activityBar.background": "#FBE7B2",
-    "activityBar.foreground": "#52358C",
-    "activityBar.inactiveForeground": "#616161",
-    "activityBar.activeBorder": "#04184d",
-    "activityBar.activeBackground": "#C3B48B",
-    "activityBar.border": "#C3B48B",
-    "titleBar.activeBackground": "#D2BE88",
-    "titleBar.activeForeground": "#52358C",
-    "titleBar.inactiveBackground": "#bdb59c",
-    "titleBar.inactiveForeground": "#616161",
-    "titleBar.border": "#C3B48B",
-    "statusBar.background": "#E9DBB7",
-    "statusBar.foreground": "#52358C",
-    "statusBar.border": "#C3B48B"
-  }
-}

package/test/test-reporter.mjs

@@ -1,68 +0,0 @@
-function colorize (type, text) {
-  if (type === 'pass') {
-    const blackText = `\x1b[30m${text}`
-    const boldblackText = `\x1b[1m${blackText}`
-    // Green background with black text
-    return `\x1b[42m${boldblackText}\x1b[0m`
-  }
-
-  if (type === 'fail') {
-    const whiteText = `\x1b[37m${text}`
-    const boldWhiteText = `\x1b[1m${whiteText}`
-    // Red background with white text
-    return `\x1b[41m${boldWhiteText}\x1b[0m`
-  }
-
-  return text
-}
-
-function formatDiagnosticStr (str) {
-  return str.replace(/^(\w+)(\s*\d*)/i, (_, firstWord, rest) => {
-    return firstWord.charAt(0).toUpperCase() + firstWord.slice(1).toLowerCase() + ':' + rest
-  })
-}
-
-async function * reporter (source) {
-  const failed = new Set()
-  const diagnostics = new Set()
-
-  for await (const event of source) {
-    switch (event.type) {
-      case 'test:pass': {
-        yield `${colorize('pass', 'PASSED')}: ${event.data.file || event.data.name}\n`
-        break
-      }
-
-      case 'test:fail': {
-        failed.add(event.data.name || event.data.file)
-        yield `${colorize('fail', 'FAILED')}: ${event.data.file || event.data.name}\n`
-        break
-      }
-
-      case 'test:diagnostic': {
-        diagnostics.add(`${formatDiagnosticStr(event.data.message)}\n`)
-        break
-      }
-
-      default: {
-        yield ''
-      }
-    }
-  }
-
-  if (failed.size > 0) {
-    yield `\n\n${colorize('fail', 'Failed tests:')}\n`
-    for (const file of failed) {
-      yield `${file}\n`
-    }
-  }
-
-  yield '\n'
-
-  for (const diagnostic of diagnostics) {
-    yield `${diagnostic}`
-  }
-  yield '\n'
-}
-
-export default reporter