Skip to content

Loading…

Update to make sure hostName exists before trying to act on it #101

Merged
merged 1 commit into from

2 participants

@GCheung55

Otherwise it will throw an error.

@ciaranj ciaranj closed this
@ciaranj ciaranj reopened this
@ciaranj ciaranj merged commit 3a15f03 into ciaranj:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jun 11, 2012
  1. @GCheung55

    Update to make sure hostName exists before trying to act on it. Other…

    GCheung55 committed
    …wise it will throw an error.
Showing with 1 addition and 1 deletion.
  1. +1 −1 lib/_utils.js
View
2 lib/_utils.js
@@ -1,4 +1,4 @@
// Returns true if this is a host that closes *before* it ends?!?!
module.exports.isAnEarlyCloseHost= function( hostName ) {
- return hostName.match(".*google(apis)?.com$")
+ return hostName && hostName.match(".*google(apis)?.com$")
}
Something went wrong with that request. Please try again.