issue-813 -- Adds enforce-domains flag to no-url-protocols rule

sasstools · Aug 25, 2016 · a120002 · a120002
1 parent 54c25a9
commit a120002
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/lib/rules/no-url-protocols.js b/lib/rules/no-url-protocols.js
@@ -2,24 +2,29 @@
 
 var helpers = require('../helpers');
 
-var isUrlRegex = /^(https?:)?\/\//;
+var isUrlRegex = /^(https?:)?\/\//,
+    noProtocolRegex = /^(https?:)\/\//;
 
 var stripQuotes = function (str) {
   return str.substring(1, str.length - 1);
 };
 
 module.exports = {
   'name': 'no-url-protocols',
-  'defaults': {},
+  'defaults': {
+    'enforce-domains' : true
+  },
   'detect': function (ast, parser) {
     var result = [];
 
     ast.traverseByType('uri', function (uri) {
       uri.traverse(function (item) {
         if (item.is('string')) {
-          var stripped = stripQuotes(item.content);
+          var stripped = stripQuotes(item.content),
+              regexSelector = parser.options['enforce-domains'] ?
+                  isUrlRegex : noProtocolRegex;
 
-          if (stripped.match(isUrlRegex)) {
+          if (stripped.match(regexSelector)) {
             result = helpers.addUnique(result, {
               'ruleId': parser.rule.name,
               'severity': parser.severity,