lint: use standard style

Author: dougwilson

.eslintignore

@@ -0,0 +1,2 @@
+coverage
+node_modules

.eslintrc

@@ -0,0 +1,3 @@
+{
+  "extends": "standard"
+}

.travis.yml

@@ -19,7 +19,7 @@ before_install:
   # Setup Node.js version-specific dependencies
   - "test $TRAVIS_NODE_VERSION != '0.6' || npm rm --save-dev istanbul"
   - "test $TRAVIS_NODE_VERSION != '0.8' || npm rm --save-dev istanbul"
-
+  - "test $(echo $TRAVIS_NODE_VERSION | cut -d. -f1) -ge 4 || npm rm --save-dev eslint eslint-config-standard eslint-plugin-promise eslint-plugin-standard"
   # Update Node.js modules
   - "test ! -d node_modules || npm prune"
   - "test ! -d node_modules || npm rebuild"

index.js

@@ -24,7 +24,7 @@ module.exports = auth
  * @private
  */
 
-var credentialsRegExp = /^ *(?:[Bb][Aa][Ss][Ii][Cc]) +([A-Za-z0-9\-\._~\+\/]+=*) *$/
+var CREDENTIALS_REGEXP = /^ *(?:[Bb][Aa][Ss][Ii][Cc]) +([A-Za-z0-9._~+/-]+=*) *$/
 
 /**
  * RegExp for basic auth user/pass
@@ -35,7 +35,7 @@ var credentialsRegExp = /^ *(?:[Bb][Aa][Ss][Ii][Cc]) +([A-Za-z0-9\-\._~\+\/]+=*)
  * @private
  */
 
-var userPassRegExp = /^([^:]*):(.*)$/
+var USER_PASS_REGEXP = /^([^:]*):(.*)$/
 
 /**
  * Parse the Authorization header field of a request.
@@ -45,7 +45,7 @@ var userPassRegExp = /^([^:]*):(.*)$/
  * @public
  */
 
-function auth(req) {
+function auth (req) {
   if (!req) {
     throw new TypeError('argument req is required')
   }
@@ -58,14 +58,14 @@ function auth(req) {
   var header = getAuthorization(req.req || req)
 
   // parse header
-  var match = credentialsRegExp.exec(header || '')
+  var match = CREDENTIALS_REGEXP.exec(header || '')
 
   if (!match) {
     return
   }
 
   // decode user pass
-  var userPass = userPassRegExp.exec(decodeBase64(match[1]))
+  var userPass = USER_PASS_REGEXP.exec(decodeBase64(match[1]))
 
   if (!userPass) {
     return
@@ -80,7 +80,7 @@ function auth(req) {
  * @private
  */
 
-function decodeBase64(str) {
+function decodeBase64 (str) {
   return new Buffer(str, 'base64').toString()
 }
 
@@ -89,7 +89,7 @@ function decodeBase64(str) {
  * @private
  */
 
-function getAuthorization(req) {
+function getAuthorization (req) {
   if (!req.headers || typeof req.headers !== 'object') {
     throw new TypeError('argument req is required to have headers property')
   }
@@ -102,7 +102,7 @@ function getAuthorization(req) {
  * @private
  */
 
-function Credentials(name, pass) {
+function Credentials (name, pass) {
   this.name = name
   this.pass = pass
 }

package.json

@@ -11,6 +11,10 @@
   ],
   "repository": "jshttp/basic-auth",
   "devDependencies": {
+    "eslint": "3.10.2",
+    "eslint-config-standard": "6.2.1",
+    "eslint-plugin-promise": "3.4.0",
+    "eslint-plugin-standard": "2.0.1",
     "istanbul": "0.4.5",
     "mocha": "1.21.5"
   },
@@ -23,6 +27,7 @@
     "node": ">= 0.6"
   },
   "scripts": {
+    "lint": "eslint .",
     "test": "mocha --check-leaks --reporter spec --bail",
     "test-cov": "istanbul cover node_modules/mocha/bin/_mocha -- --reporter dot --check-leaks test/",
     "test-travis": "istanbul cover node_modules/mocha/bin/_mocha --report lcovonly -- --reporter spec --check-leaks test/"

test/.eslintrc

@@ -0,0 +1,5 @@
+{
+  "env": {
+    "mocha": true
+  }
+}

test/basic-auth.js

@@ -1,7 +1,7 @@
-var assert = require('assert');
-var auth = require('..');
+var assert = require('assert')
+var auth = require('..')
 
-function request(authorization) {
+function request (authorization) {
   return {
     headers: {
       authorization: authorization
@@ -44,76 +44,76 @@ describe('auth(req)', function () {
     })
   })
 
-  describe('with no Authorization field', function(){
-    it('should return null', function(){
-      var req = request();
-      assert(null == auth(req));
+  describe('with no Authorization field', function () {
+    it('should return null', function () {
+      var req = request()
+      assert(auth(req) == null)
     })
   })
 
-  describe('with malformed Authorization field', function(){
-    it('should return null', function(){
-      var req = request('Something');
-      assert(null == auth(req));
+  describe('with malformed Authorization field', function () {
+    it('should return null', function () {
+      var req = request('Something')
+      assert(auth(req) == null)
     })
   })
 
-  describe('with malformed Authorization scheme', function(){
-    it('should return null', function(){
-      var req = request('basic_Zm9vOmJhcg==');
-      assert(null == auth(req));
+  describe('with malformed Authorization scheme', function () {
+    it('should return null', function () {
+      var req = request('basic_Zm9vOmJhcg==')
+      assert(auth(req) == null)
     })
   })
 
-  describe('with malformed credentials', function(){
-    it('should return null', function(){
-      var req = request('basic Zm9vcgo=');
-      assert(null == auth(req));
+  describe('with malformed credentials', function () {
+    it('should return null', function () {
+      var req = request('basic Zm9vcgo=')
+      assert(auth(req) == null)
     })
   })
 
-  describe('with valid credentials', function(){
-    it('should return .name and .pass', function(){
-      var req = request('basic Zm9vOmJhcg==');
-      var creds = auth(req);
-      assert.equal(creds.name, 'foo');
-      assert.equal(creds.pass, 'bar');
+  describe('with valid credentials', function () {
+    it('should return .name and .pass', function () {
+      var req = request('basic Zm9vOmJhcg==')
+      var creds = auth(req)
+      assert.equal(creds.name, 'foo')
+      assert.equal(creds.pass, 'bar')
     })
   })
 
-  describe('with empty password', function(){
-    it('should return .name and .pass', function(){
-      var req = request('basic Zm9vOg==');
-      var creds = auth(req);
-      assert.equal(creds.name, 'foo');
-      assert.equal(creds.pass, '');
+  describe('with empty password', function () {
+    it('should return .name and .pass', function () {
+      var req = request('basic Zm9vOg==')
+      var creds = auth(req)
+      assert.equal(creds.name, 'foo')
+      assert.equal(creds.pass, '')
     })
   })
 
-  describe('with empty userid', function(){
-    it('should return .name and .pass', function(){
-      var req = request('basic OnBhc3M=');
-      var creds = auth(req);
-      assert.equal(creds.name, '');
-      assert.equal(creds.pass, 'pass');
+  describe('with empty userid', function () {
+    it('should return .name and .pass', function () {
+      var req = request('basic OnBhc3M=')
+      var creds = auth(req)
+      assert.equal(creds.name, '')
+      assert.equal(creds.pass, 'pass')
     })
   })
 
-  describe('with empty userid and pass', function(){
-    it('should return .name and .pass', function(){
-      var req = request('basic Og==');
-      var creds = auth(req);
-      assert.equal(creds.name, '');
-      assert.equal(creds.pass, '');
+  describe('with empty userid and pass', function () {
+    it('should return .name and .pass', function () {
+      var req = request('basic Og==')
+      var creds = auth(req)
+      assert.equal(creds.name, '')
+      assert.equal(creds.pass, '')
     })
   })
 
-  describe('with colon in pass', function(){
-    it('should return .name and .pass', function(){
-      var req = request('basic Zm9vOnBhc3M6d29yZA==');
-      var creds = auth(req);
-      assert.equal(creds.name, 'foo');
-      assert.equal(creds.pass, 'pass:word');
+  describe('with colon in pass', function () {
+    it('should return .name and .pass', function () {
+      var req = request('basic Zm9vOnBhc3M6d29yZA==')
+      var creds = auth(req)
+      assert.equal(creds.name, 'foo')
+      assert.equal(creds.pass, 'pass:word')
     })
   })
 })