why instantiate a new RegExp object each time? You should be able to do `!ipRegex.test(ip)`.