Support commas in URLs (#76)

* commas should also be supported in URLs * Update URL validation regex: commas should be available characters * Update URL validation regex: braces should also be available characters
claudiajs · Jan 6, 2017 · 7a2eb92 · 7a2eb92
1 parent 0abb765
commit 7a2eb92
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/is-url.js b/lib/is-url.js
@@ -1,6 +1,6 @@
 'use strict';
 
 module.exports = function isUrl(url) {
-  const pattern = /^[-a-zA-Z0-9@:%_\+.~#?&//=]{2,256}\.[a-z]{2,63}\b(\/[-a-zA-Z0-9@:%_\+.~#?&//=]*)?$/gi;
+  const pattern = /^[-a-zA-Z0-9@:%_\+.~#?&//=]{2,256}\.[a-z]{2,63}\b(\/[-a-zA-Z0-9@:%_\(\)\+.,~#?&//=]*)?$/gi;
   return pattern.test(url);
 };
diff --git a/spec/is-url-spec.js b/spec/is-url-spec.js
@@ -16,6 +16,7 @@ describe('isUrl', () => {
     expect(isUrl('https://www.claudiajs.com')).toBeTruthy();
     expect(isUrl('https://github.com/claudiajs')).toBeTruthy();
     expect(isUrl('https://github.com/claudiajs/claudia-bot-builder')).toBeTruthy();
+    expect(isUrl('https://www.google.com/#q=claudia,bot')).toBeTruthy();
   });
   it('should return false if url is in the sentence', () => {
     expect(isUrl('This is a valid url: http://google.com')).toBeFalsy();