bare-http1 4.1.2 → 4.1.3

package/index.d.ts

@@ -8,11 +8,7 @@ import {
 } from 'bare-tcp'
 import Buffer from 'bare-buffer'
 import URL from 'bare-url'
-import constants, {
-  HTTPMethod,
-  HTTPStatusCode,
-  HTTPStatusMessage
-} from './lib/constants'
+import constants, { HTTPMethod, HTTPStatusCode, HTTPStatusMessage } from './lib/constants'
 import HTTPError from './lib/errors'
 
 export {
@@ -111,18 +107,13 @@ export interface HTTPAgent {
 
   getName(opts: { host: string; port: number }): string
 
-  addRequest(
-    req: HTTPClientRequest,
-    opts: TCPSocketOptions & TCPSocketConnectOptions
-  ): void
+  addRequest(req: HTTPClientRequest, opts: TCPSocketOptions & TCPSocketConnectOptions): void
 
   destroy(): void
 }
 
 export class HTTPAgent {
-  constructor(
-    opts?: HTTPAgentOptions & TCPSocketOptions & TCPSocketConnectOptions
-  )
+  constructor(opts?: HTTPAgentOptions & TCPSocketOptions & TCPSocketConnectOptions)
 
   static global: HTTPAgent
 }
@@ -137,8 +128,7 @@ export interface HTTPServerEvents extends TCPServerEvents {
   timeout: [socket: TCPSocket]
 }
 
-export interface HTTPServer<M extends HTTPServerEvents = HTTPServerEvents>
-  extends TCPServer<M> {
+export interface HTTPServer<M extends HTTPServerEvents = HTTPServerEvents> extends TCPServer<M> {
   readonly timeout: number | undefined
 
   setTimeout(ms: number, ontimeout?: () => void): this
@@ -150,9 +140,7 @@ export class HTTPServer {
     onrequest?: (req: HTTPIncomingMessage, res: HTTPServerResponse) => void
   )
 
-  constructor(
-    onrequest: (req: HTTPIncomingMessage, res: HTTPServerResponse) => void
-  )
+  constructor(onrequest: (req: HTTPIncomingMessage, res: HTTPServerResponse) => void)
 }
 
 export { HTTPServer as Server }
@@ -168,10 +156,7 @@ export interface HTTPServerResponse extends HTTPOutgoingMessage {
     headers?: Record<string, string | number>
   ): void
 
-  writeHead(
-    statusCode: HTTPStatusCode,
-    headers?: Record<string, string | number>
-  ): void
+  writeHead(statusCode: HTTPStatusCode, headers?: Record<string, string | number>): void
 }
 
 export class HTTPServerResponse {
@@ -194,11 +179,7 @@ export interface HTTPServerConnection {
 }
 
 export class HTTPServerConnection {
-  constructor(
-    server: HTTPServer,
-    socket: TCPSocket,
-    opts?: HTTPServerConnectionOptions
-  )
+  constructor(server: HTTPServer, socket: TCPSocket, opts?: HTTPServerConnectionOptions)
 
   static for(socket: TCPSocket): HTTPServerConnection
 }
@@ -217,9 +198,8 @@ export interface HTTPClientRequestOptions extends TCPSocketConnectOptions {
   path?: string
 }
 
-export interface HTTPClientRequest<
-  M extends HTTPClientRequestEvents = HTTPClientRequestEvents
-> extends HTTPOutgoingMessage<M> {
+export interface HTTPClientRequest<M extends HTTPClientRequestEvents = HTTPClientRequestEvents>
+  extends HTTPOutgoingMessage<M> {
   readonly method: HTTPMethod
   readonly path: string
   readonly headers: Record<string, string | number>
@@ -251,10 +231,7 @@ export class HTTPClientConnection {
 
   static for(socket: TCPSocket): HTTPClientConnection | null
 
-  static from(
-    socket: TCPSocket,
-    opts?: HTTPClientConnectionOptions
-  ): HTTPClientConnection
+  static from(socket: TCPSocket, opts?: HTTPClientConnectionOptions): HTTPClientConnection
 }
 
 export { HTTPClientConnection as ClientConnection }

package/index.js

@@ -40,10 +40,12 @@ exports.request = function request(url, opts, onresponse) {
 
     // For Node.js compatibility
     opts.host = opts.hostname || opts.host
-    opts.port =
-      typeof opts.port === 'string' ? parseInt(opts.port, 10) : opts.port
+    opts.port = typeof opts.port === 'string' ? parseInt(opts.port, 10) : opts.port
   }
 
+  // TODO: Renable the default global agent when hangs have been sorted
+  opts.agent = opts.agent || new exports.Agent()
+
   return new exports.ClientRequest(opts, onresponse)
 }
 

package/lib/agent.js

@@ -12,12 +12,7 @@ module.exports = class HTTPAgent extends EventEmitter {
     this._sockets = new Map()
     this._freeSockets = new Map()
 
-    this._keepAlive =
-      typeof keepAlive === 'number'
-        ? keepAlive
-        : keepAlive
-          ? keepAliveMsecs
-          : -1
+    this._keepAlive = typeof keepAlive === 'number' ? keepAlive : keepAlive ? keepAliveMsecs : -1
 
     this._opts = { ...opts }
   }
@@ -108,9 +103,7 @@ module.exports = class HTTPAgent extends EventEmitter {
   }
 
   _onremove(socket, name, all = true) {
-    for (const set of all
-      ? [this._sockets, this._freeSockets]
-      : [this._sockets]) {
+    for (const set of all ? [this._sockets, this._freeSockets] : [this._sockets]) {
       const sockets = set.get(name)
       if (sockets === undefined) continue
 

package/lib/client-connection.js

@@ -90,10 +90,7 @@ module.exports = class HTTPClientConnection {
               this.socket.emit('free')
             })
 
-            if (
-              op.headers.connection &&
-              op.headers.connection.toLowerCase() === 'upgrade'
-            ) {
+            if (op.headers.connection && op.headers.connection.toLowerCase() === 'upgrade') {
               return this._onupgrade(this._parser.end())
             }
 

package/lib/client-request.js

@@ -13,8 +13,7 @@ module.exports = class HTTPClientRequest extends HTTPOutgoingMessage {
 
     opts = opts ? { ...opts } : {}
 
-    const agent =
-      opts.agent === false ? new HTTPAgent() : opts.agent || HTTPAgent.global
+    const agent = opts.agent === false ? new HTTPAgent() : opts.agent || HTTPAgent.global
     const method = opts.method || 'GET'
     const path = opts.path || '/'
     const host = (opts.host = opts.host || 'localhost')
@@ -50,8 +49,7 @@ module.exports = class HTTPClientRequest extends HTTPOutgoingMessage {
       const v = this.headers[name]
 
       if (n === 'content-length') this._chunked = false
-      if (n === 'connection' && v && v.toLowerCase() === 'upgrade')
-        upgrade = true
+      if (n === 'connection' && v && v.toLowerCase() === 'upgrade') upgrade = true
 
       h += `${httpCase(n)}: ${v}\r\n`
     }

package/lib/server-connection.js

@@ -18,10 +18,7 @@ module.exports = class HTTPServerConnection {
   }
 
   constructor(server, socket, opts = {}) {
-    const {
-      IncomingMessage = HTTPIncomingMessage,
-      ServerResponse = HTTPServerResponse
-    } = opts
+    const { IncomingMessage = HTTPIncomingMessage, ServerResponse = HTTPServerResponse } = opts
 
     this.server = server
     this.socket = socket
@@ -89,10 +86,7 @@ module.exports = class HTTPServerConnection {
             this.req.resume()
             this.req.pause()
 
-            if (
-              op.headers.connection &&
-              op.headers.connection.toLowerCase() === 'upgrade'
-            ) {
+            if (op.headers.connection && op.headers.connection.toLowerCase() === 'upgrade') {
               return this._onupgrade(this._parser.end())
             }
 

package/lib/server-response.js

@@ -42,9 +42,7 @@ module.exports = class HTTPServerResponse extends HTTPOutgoingMessage {
       'HTTP/1.1 ' +
       this.statusCode +
       ' ' +
-      (this.statusMessage === null
-        ? constants.status[this.statusCode]
-        : this.statusMessage) +
+      (this.statusMessage === null ? constants.status[this.statusCode] : this.statusMessage) +
       '\r\n'
 
     for (const name of Object.keys(this.headers)) {
@@ -52,8 +50,7 @@ module.exports = class HTTPServerResponse extends HTTPOutgoingMessage {
       const v = this.headers[name]
 
       if (n === 'content-length') this._chunked = false
-      if (n === 'connection' && v && v.toLowerCase() === 'close')
-        this._close = true
+      if (n === 'connection' && v && v.toLowerCase() === 'close') this._close = true
 
       h += httpCase(n) + ': ' + v + '\r\n'
     }

package/lib/server.js

@@ -12,10 +12,7 @@ module.exports = class HTTPServer extends TCPServer {
 
     this._timeout = 0
 
-    this.on(
-      'connection',
-      (socket) => new HTTPServerConnection(this, socket, opts)
-    )
+    this.on('connection', (socket) => new HTTPServerConnection(this, socket, opts))
 
     if (onrequest) this.on('request', onrequest)
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bare-http1",
-  "version": "4.1.2",
+  "version": "4.1.3",
   "description": "Native HTTP/1 library for JavaScript",
   "exports": {
     "./package": "./package.json",
@@ -46,7 +46,7 @@
     "bare-url": "^2.1.3",
     "brittle": "^3.3.0",
     "prettier": "^3.4.1",
-    "prettier-config-standard": "^7.0.0"
+    "prettier-config-holepunch": "^2.0.0"
   },
   "peerDependencies": {
     "bare-buffer": "*",