Bug in isStringArray

[artola]

I was reading the code to evaluate the usage, I found this line:

http2-express-bridge/src/util.js

Line 7 in 76b699e

return false

The problem is that the return false does nothing more than finish the iteration and moving to the next element of the array.

    if (Array.isArray(arr)) {

        arr.forEach(item => {
           if(typeof item !== 'string'){
              return false  // <======= here
           }
        })

        return true
    }

Probably it was intended this optimised version that will stop asap:

    if (Array.isArray(arr)) {

        return arr.every(item => typeof item === 'string')

    }