fix: make feature strings more robust to whitespace (#15602)

Fixes #15594

lib/common/parse-features-string.js

@@ -4,12 +4,12 @@
 // - `features` input string
 // - `emit` function(key, value) - called for each parsed KV
 module.exports = function parseFeaturesString (features, emit) {
-  features = `${features}`
+  features = `${features}`.trim()
   // split the string by ','
-  features.split(/,\s*/).forEach((feature) => {
+  features.split(/\s*,\s*/).forEach((feature) => {
     // expected form is either a key by itself or a key/value pair in the form of
     // 'key=value'
-    let [key, value] = feature.split(/\s*=/)
+    let [key, value] = feature.split(/\s*=\s*/)
     if (!key) return
 
     // interpret the value as a boolean, if possible

spec/internal-spec.js

@@ -0,0 +1,23 @@
+const chai = require('chai')
+const { expect } = chai
+
+describe('feature-string parsing', () => {
+  it('is indifferent to whitespace around keys and values', () => {
+    const parseFeaturesString = require('@electron/internal/common/parse-features-string')
+    const checkParse = (string, parsed) => {
+      const features = {}
+      parseFeaturesString(string, (k, v) => { features[k] = v })
+      expect(features).to.deep.equal(parsed)
+    }
+    checkParse('a=yes,c=d', { a: true, c: 'd' })
+    checkParse('a=yes ,c=d', { a: true, c: 'd' })
+    checkParse('a=yes, c=d', { a: true, c: 'd' })
+    checkParse('a=yes , c=d', { a: true, c: 'd' })
+    checkParse(' a=yes , c=d', { a: true, c: 'd' })
+    checkParse(' a= yes , c=d', { a: true, c: 'd' })
+    checkParse(' a = yes , c=d', { a: true, c: 'd' })
+    checkParse(' a = yes , c =d', { a: true, c: 'd' })
+    checkParse(' a = yes , c = d', { a: true, c: 'd' })
+    checkParse(' a = yes , c = d ', { a: true, c: 'd' })
+  })
+})