Revert "Remove excess code"

This reverts commit b3e4bf7.

Author: s0ph1e

lib/config/defaults.js

@@ -1,5 +1,6 @@
 var config = {
   defaultFilename: 'index.html',
+  log: false,
   sources: [
     {
       selector: 'img',
@@ -32,7 +33,7 @@ var config = {
     {
       selector: 'link[rel*="icon"]',
       attr: 'href'
-    }
+    },
   ],
   subdirectories: [
     {

lib/utils.js

@@ -1,9 +1,14 @@
 var url = require('url');
 var path = require('path');
 
+function isUrl(path) {
+	var urlRegexp = /^((http[s]?:)?\/\/)/;
+	return urlRegexp.test(path);
+}
+
 function getUrl(currentUrl, path) {
 	var pathObj = url.parse(path);
-	if (!pathObj.protocol) {
+	if (isUrl(path) && !pathObj.protocol) {
 		pathObj.protocol = 'http';
 		path = url.format(pathObj);
 	}
@@ -25,6 +30,7 @@ function getFilenameFromUrl (u) {
 }
 
 module.exports = {
+	isUrl: isUrl,
 	getUrl: getUrl,
 	getUnixPath: getUnixPath,
 	getRelativePath: getRelativePath,

test/unit/utils-test.js

@@ -2,6 +2,23 @@ require('should');
 var utils = require('../../lib/utils');
 
 describe('Common utils', function () {
+	describe('#isUrl(url)', function () {
+		it('should return true if url starts with "http[s]://"', function () {
+			utils.isUrl('http://google.com').should.be.true();
+			utils.isUrl('https://github.com').should.be.true();
+		});
+		it('should return true if url starts with "//"', function () {
+			utils.isUrl('//www.youtube.com').should.be.true();
+		});
+		it('should return false if url starts neither with "http[s]://" nor "//"', function () {
+			utils.isUrl('http//www.youtube.com').should.be.false();
+			utils.isUrl('http:/www.youtube.com').should.be.false();
+			utils.isUrl('htt://www.youtube.com').should.be.false();
+			utils.isUrl('://www.youtube.com').should.be.false();
+			utils.isUrl('www.youtube.com').should.be.false();
+		});
+	});
+
 	describe('#getUrl(url, path)', function () {
 		it('should return url + path if path is not url', function () {
 			utils.getUrl('http://google.com', '/path').should.be.equal('http://google.com/path');