Fixes URLs with query strings and hashes in them

The previous fix did not test a corner case where
the user passes extensions allowed in the options.
When there are no otpions passed for extensions
allowed the extension check is skipped and thus the
file is processed appropriately.

I added virtual copies of the tests so feel free to
point me to a better way of doing that. I just don't
know what a better way to do it is without overwriting
the original tests.

The fix I made was stripping query strings and hashes
from the regex results before the extension check and
then passing in that stripped version into the check
instead of the actual result which has the hash and/or
query string.

Author: Gedalia Kott

src/index.js

@@ -58,7 +58,8 @@ function gulpCssBase64(opts) {
                     return;
                   }
 
-                  if (opts.extensionsAllowed.length !== 0 && opts.extensionsAllowed.indexOf(path.extname(result[1])) === -1) {
+                  var pureURL = stripQueryStringFromURL(stripHashFromURL(result[1]));
+                  if (opts.extensionsAllowed.length !== 0 && opts.extensionsAllowed.indexOf(path.extname(pureURL)) === -1) {
                     log('Ignores ' + chalk.yellow(result[1]) + ', extension not allowed ' + chalk.yellow(path.extname(result[1])), opts.verbose);
                     callback();
                     return;
@@ -190,5 +191,13 @@ function log(message, isVerbose) {
   }
 }
 
+function stripQueryStringFromURL(urlString) {
+  return urlString.split('?')[0];
+}
+
+function stripHashFromURL(urlString) {
+  return urlString.split('#')[0];
+}
+
 // Exporting the plugin main function
 module.exports = gulpCssBase64;