Ensure always provide a buffer as input

Author: Kikobeats

index.js

@@ -1,17 +1,20 @@
 'use strict'
 
 var jschardet = require('jschardet')
+var isBuffer = require('is-buffer')
 var iconv = require('iconv-lite')
 var charset = require('charset')
 
 var charsetRegex = /charset=["]*([^>"\s]+)/i
 
-module.exports = function ensureUTF8 (binaryBuffer, contentType) {
-  var encoding = getEncoding(binaryBuffer, contentType)
+module.exports = function ensureUTF8 (buffer, contentType) {
+  if (!isBuffer(buffer)) throw new TypeError('content should be a buffer.')
+
+  var encoding = getEncoding(buffer, contentType)
 
   return encoding === 'utf8'
-    ? binaryBuffer.toString('utf8')
-    : iconv.decode(binaryBuffer, encoding).replace(charsetRegex, 'utf-8')
+    ? buffer.toString('utf8')
+    : iconv.decode(buffer, encoding).replace(charsetRegex, 'utf-8')
 }
 
 function getEncoding (content, contentType) {

package.json

@@ -23,7 +23,9 @@
   "dependencies": {
     "charset": "~1.0.1",
     "iconv-lite": "~0.4.19",
-    "jschardet": "~1.6.0"
+    "is-buffer": "~1.1.6",
+    "jschardet": "~1.6.0",
+    "to-buffer": "~1.1.0"
   },
   "devDependencies": {
     "coveralls": "latest",

test.js

@@ -0,0 +1,11 @@
+'use strict'
+
+var iconv = require('iconv-lite')
+var path = require('path')
+var fs = require('fs')
+var toUTF8 = require('./')
+
+const buffer = fs.readFileSync(path.join(__dirname, 'test/fixtures/rp.pl.html'))
+
+var str = toUTF8(buffer, 'windows-1250')
+console.log(str.includes('majątków'))