Merge pull request #1067 from Jonnymcc/fix-parse-ini-comments

Parse ini files with comments starting with #

Author: chrisradek

.changes/next-release/feature-util-5a565afe.json

@@ -0,0 +1,5 @@
+{
+  "type": "feature",
+  "category": "util",
+  "description": "Parse ini files containing comments using #"
+}

lib/util.js

@@ -189,7 +189,7 @@ var util = {
     parse: function string(ini) {
       var currentSection, map = {};
       util.arrayEach(ini.split(/\r?\n/), function(line) {
-        line = line.split(/(^|\s);/)[0]; // remove comments
+        line = line.split(/(^|\s)[;#]/)[0]; // remove comments
         var section = line.match(/^\s*\[([^\[\]]+)\]\s*$/);
         if (section) {
           currentSection = section[1];

test/util.spec.coffee

@@ -207,11 +207,14 @@ describe 'AWS.util.ini', ->
       invalidline
       key1=value1 ; another comment
         key2 = value2;value3
+        key3 = value4 # yet another comment
       [emptysection]
+      #key1=value1
       '''
       map = AWS.util.ini.parse(ini)
       expect(map.section1.key1).to.equal('value1')
       expect(map.section1.key2).to.equal('value2;value3')
+      expect(map.section1.key3).to.equal('value4')
       expect(map.emptysection).to.equal(undefined)
 
     it 'ignores leading and trailing white space', ->