bare-http1 3.3.1 → 3.4.0

package/README.md

@@ -6,9 +6,9 @@ HTTP/1 library for JavaScript.
 npm i bare-http1
 ```
 
-Currently HTTP servers does NOT support server request bodies, but supports most other HTTP features (keep-alive, chunked encoding, etc.) and streaming server responses.
+`http.Server` supports most HTTP features, e.g. keep-alive, chunked encoding, and streaming responses.
 
-Basic HTTP client is supported, but currently it does NOT support keep-alive and protocol negotiation.
+Basic `http.ClientRequest` is implemented, temporarily it does NOT support keep-alive and protocol negotiation.
 
 ## Usage
 

package/lib/client-connection.js

@@ -178,6 +178,6 @@ module.exports = class HTTPClientConnection {
   }
 
   _ondrain () {
-    if (this.res) this.res._continueWrite()
+    if (this.req) this.req._continueWrite()
   }
 }

package/lib/client-request.js

@@ -22,9 +22,10 @@ module.exports = class HTTPClientRequest extends HTTPOutgoingMessage {
     const host = opts.host || 'localhost'
     const port = opts.port || 80
 
-    this.headers = { host: host + ':' + port }
+    this.headers = { host: host + ':' + port, ...opts.headers }
 
     this._connection = connection
+    this._chunked = true
 
     this._pendingFinal = null
 
@@ -38,20 +39,48 @@ module.exports = class HTTPClientRequest extends HTTPOutgoingMessage {
       const n = name.toLowerCase()
       const v = this.headers[name]
 
+      if (n === 'content-length') this._chunked = false
+
       h += `${httpCase(n)}: ${v}\r\n`
     }
 
+    if (this._chunked) h += 'Transfer-Encoding: chunked\r\n'
+
     h += '\r\n'
 
     return h
   }
 
+  _write (data, cb) {
+    if (this.headersSent === false) this.flushHeaders()
+
+    if (this._chunked) {
+      data = Buffer.concat([
+        Buffer.from('' + data.byteLength.toString(16) + '\r\n'),
+        data,
+        Buffer.from('\r\n')
+      ])
+    }
+
+    if (this.socket.write(data)) cb(null)
+    else this._pendingWrite = cb
+  }
+
   _final (cb) {
     if (this.headersSent === false) this.flushHeaders()
 
+    if (this._chunked) this.socket.write(Buffer.from('0\r\n\r\n'))
+
     this._pendingFinal = cb
   }
 
+  _continueWrite () {
+    if (this._pendingWrite === null) return
+    const cb = this._pendingWrite
+    this._pendingWrite = null
+    cb(null)
+  }
+
   _continueFinal () {
     if (this._pendingFinal === null) return
     const cb = this._pendingFinal

package/lib/server-connection.js

@@ -34,6 +34,8 @@ module.exports = class HTTPServerConnection {
   }
 
   _ondata (data) {
+    if (this._state === constants.state.IN_BODY) return this._onbody(data)
+
     if (this._buffer !== null) {
       this._buffer = Buffer.concat([this._buffer, data])
     } else {
@@ -49,6 +51,30 @@ module.exports = class HTTPServerConnection {
         hits++
       } else if (hits === 1 && b === 10) {
         hits++
+
+        if (this._state === constants.state.BEFORE_CHUNK) {
+          const head = this._buffer.subarray(0, i - 1)
+          this._buffer = i + 1 === this._buffer.byteLength ? null : this._buffer.subarray(i + 1)
+          i = 0
+          hits = 0
+          this._onchunklength(head)
+
+          if (this._buffer === null) break
+        } else if (this._state === constants.state.IN_CHUNK) {
+          const chunk = this._buffer.subarray(0, i - 1)
+
+          if (chunk.byteLength !== this._length) {
+            hits = 0
+            continue
+          }
+
+          this._buffer = i + 1 === this._buffer.byteLength ? null : this._buffer.subarray(i + 1)
+          i = 0
+          hits = 0
+          this._onchunk(chunk)
+
+          if (this._buffer === null) break
+        }
       } else if (hits === 2 && b === 13) {
         hits++
       } else if (hits === 3 && b === 10) {
@@ -86,12 +112,53 @@ module.exports = class HTTPServerConnection {
     this.req = new this._IncomingMessage(this.socket, headers, { method, url })
     this.res = new this._ServerResponse(this.socket, this.req, headers.connection === 'close')
 
-    this.req.on('close', () => { this.req = null })
-    this.res.on('close', () => { this.res = null; this._onreset() })
-
-    this.req.push(null)
+    this.res.on('close', () => { this.res = null })
+    this.req.on('close', () => { this.req = null; this._onreset() })
 
     this.server.emit('request', this.req, this.res)
+
+    if (headers['transfer-encoding'] === 'chunked') {
+      this._state = constants.state.BEFORE_CHUNK
+    } else {
+      this._length = parseInt(headers['content-length'], 10) || 0
+
+      if (this._length === 0) return this._onfinished()
+
+      this._state = constants.state.IN_BODY
+
+      if (this._buffer) {
+        const body = this._buffer
+        this._buffer = null
+        this._onbody(body)
+      }
+    }
+  }
+
+  _onchunklength (data) {
+    this._length = parseInt(data.toString(), 16)
+
+    if (this._length === 0) this._onfinished()
+    else this._state = constants.state.IN_CHUNK
+  }
+
+  _onchunk (data) {
+    this._read += data.byteLength
+
+    this.req.push(data)
+
+    this._state = constants.state.BEFORE_CHUNK
+  }
+
+  _onbody (data) {
+    this._read += data.byteLength
+
+    this.req.push(data)
+
+    if (this._read === this._length) this._onfinished()
+  }
+
+  _onfinished () {
+    if (this.req) this.req.push(null)
   }
 
   _onreset () {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bare-http1",
-  "version": "3.3.1",
+  "version": "3.4.0",
   "description": "Native HTTP/1 library for JavaScript",
   "exports": {
     ".": "./index.js",