converted text parser to use generic parser

Author: Shawn Dellysse

lib/types/text.js

@@ -10,12 +10,8 @@
  * Module dependencies.
  */
 
-var bytes = require('bytes')
-var contentType = require('content-type')
-var debug = require('debug')('body-parser:text')
-var isFinished = require('on-finished').isFinished
-var read = require('../read')
-var typeis = require('type-is')
+var assign = require('object-assign')
+var genericParser = require('../generic-parser')
 
 /**
  * Module exports.
@@ -35,95 +31,11 @@ function text (options) {
   var opts = options || {}
 
   var defaultCharset = opts.defaultCharset || 'utf-8'
-  var inflate = opts.inflate !== false
-  var limit = typeof opts.limit !== 'number'
-    ? bytes.parse(opts.limit || '100kb')
-    : opts.limit
   var type = opts.type || 'text/plain'
-  var verify = opts.verify || false
-  var parser = opts.parser || defaultParser
 
-  if (verify !== false && typeof verify !== 'function') {
-    throw new TypeError('option verify must be function')
-  }
-
-  // create the appropriate type checking function
-  var shouldParse = typeof type !== 'function'
-    ? typeChecker(type)
-    : type
-
-  function parse (buf) {
-    return parser(buf)
-  }
-
-  return function textParser (req, res, next) {
-    if (isFinished(req)) {
-      debug('body already parsed')
-      next()
-      return
-    }
-
-    if (!('body' in req)) {
-      req.body = undefined
-    }
-
-    // skip requests without bodies
-    if (!typeis.hasBody(req)) {
-      debug('skip empty body')
-      next()
-      return
-    }
-
-    debug('content-type %j', req.headers['content-type'])
-
-    // determine if request should be parsed
-    if (!shouldParse(req)) {
-      debug('skip parsing')
-      next()
-      return
-    }
-
-    // get charset
-    var charset = getCharset(req) || defaultCharset
-
-    // read
-    read(req, res, next, parse, debug, {
-      encoding: charset,
-      inflate: inflate,
-      limit: limit,
-      verify: verify
-    })
-  }
-}
-
-function defaultParser (buf) {
-  return buf
-}
-
-/**
- * Get the charset of a request.
- *
- * @param {object} req
- * @api private
- */
-
-function getCharset (req) {
-  try {
-    return (contentType.parse(req).parameters.charset || '').toLowerCase()
-  } catch (e) {
-    return undefined
-  }
-}
-
-/**
- * Get the simple type checker.
- *
- * @param {string} type
- * @return {function}
- */
-
-function typeChecker (type) {
-  return function checkType (req) {
-    return Boolean(typeis(req, type))
-  }
+  return genericParser(assign({}, opts, {
+    type: type,
+    charset: function validateCharset () { return true },
+    defaultCharset: defaultCharset
+  }))
 }

test/text.js

@@ -20,16 +20,6 @@ describe('bodyParser.text()', function () {
       .expect(200, '"user is tobi"', done)
   })
 
-  it('should parse text/plain with a custom parser', function (done) {
-    request(createServer({
-      parser: function (input) { return input.toUpperCase() }
-    }))
-    .post('/')
-    .set('Content-Type', 'text/plain')
-    .send('user is tobi')
-    .expect(200, '"USER IS TOBI"', done)
-  })
-
   it('should 400 when invalid content-length', function (done) {
     var textParser = bodyParser.text()
     var server = createServer(function (req, res, next) {