Rescue all head errors #112

Merged
merged 1 commit into from Mar 18, 2013

Conversation

Projects
None yet
2 participants
Contributor

adarsh commented Mar 3, 2013

  • Some errors while checking a uri's head were failing due to malformed html
  • Rescuing all errors returns false for any reason, so as long as the get works, the uri will pass
Contributor

adarsh commented Mar 5, 2013

Owner

georgebrock commented Mar 18, 2013

Good catch @adarshpandit. It definitely makes sense to allow the GET fallback attempt regardless of the type of error the HEAD request throws up.

Good to merge.

Rescue all head errors
* Some errors while checking a uri's head were failing due to malformed html
* Rescuing all errors returns false for any reason, so as long as the `get` works, the uri will pass

@adarsh adarsh merged commit 1dd2ec5 into master Mar 18, 2013

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment