Fix regression accepting a Koa context

fixes #18 closes #19
jshttp · Jul 2, 2015 · c7647ca · c7647ca
1 parent d479495
commit c7647ca
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 1 deletion.
diff --git a/HISTORY.md b/HISTORY.md
@@ -1,3 +1,8 @@
+unreleased
+==========
+
+  * Fix regression accepting a Koa context
+
 1.0.2 / 2015-06-12
 ==================
 

diff --git a/index.js b/index.js
@@ -54,7 +54,6 @@ function auth(req) {
   var header = (req.req || req).headers.authorization
 
   // parse header
-  var header = req.headers.authorization
   var match = credentialsRegExp.exec(header || '')
 
   if (!match) {

diff --git a/test/basic-auth.js b/test/basic-auth.js
@@ -15,6 +15,20 @@ describe('auth(req)', function () {
       it('should be required', function () {
         assert.throws(auth, /argument req is required/)
       })
+
+      it('should accept a request', function () {
+        var req = request('basic Zm9vOmJhcg==')
+        var creds = auth(req)
+        assert.equal(creds.name, 'foo')
+        assert.equal(creds.pass, 'bar')
+      })
+
+      it('should accept a koa context', function () {
+        var ctx = { req: request('basic Zm9vOmJhcg==') }
+        var creds = auth(ctx)
+        assert.equal(creds.name, 'foo')
+        assert.equal(creds.pass, 'bar')
+      })
     })
   })