Merge pull request #361 from fardog/master

Don't require a user in the DSN

Author: dcramer

src/raven.js

@@ -423,7 +423,7 @@ function triggerEvent(eventType, options) {
 }
 
 var dsnKeys = 'source protocol user pass host port path'.split(' '),
-    dsnPattern = /^(?:(\w+):)?\/\/(\w+)(:\w+)?@([\w\.-]+)(?::(\d+))?(\/.*)/;
+    dsnPattern = /^(?:(\w+):)?\/\/(?:(\w+)(:\w+)?@)?([\w\.-]+)(?::(\d+))?(\/.*)/;
 
 function RavenConfigError(message) {
     this.name = 'RavenConfigError';

test/raven.test.js

@@ -359,6 +359,15 @@ describe('globals', function() {
             assert.strictEqual(pieces.host, 'matt-robenolt.com');
         });
 
+        it('should parse domain without user', function() {
+            var pieces = parseDSN('http://matt-robenolt.com/1');
+            assert.strictEqual(pieces.protocol, 'http');
+            assert.strictEqual(pieces.user, '');
+            assert.strictEqual(pieces.port, '');
+            assert.strictEqual(pieces.path, '/1');
+            assert.strictEqual(pieces.host, 'matt-robenolt.com');
+        });
+
         it('should raise a RavenConfigError when setting a password', function() {
             try {
                 parseDSN('http://user:[email protected]/2');