Skip to content
This repository has been archived by the owner on Feb 18, 2018. It is now read-only.

Use HTTP head instead of GET by default #11

Merged
merged 1 commit into from
Nov 5, 2011
Merged

Use HTTP head instead of GET by default #11

merged 1 commit into from
Nov 5, 2011

Commits on Nov 4, 2011

  1. Use HTTP head instead of GET by default

    This patch replaces the http GET request with HEAD. HEAD requests should
    be handled identically, but the response does not contain message
    body. GET will still be used when @get_request option is set to true.
    
    Replacing the get_response (which calls GET internally) with a new
    http HEAD request would mean code duplication. Thus, the part that does
    the actual ping is refactored to a separate private method.
    This also means that new requests resulting from redirections are
    now subject to timeout limit and they'll correctly use https.
    
    Signed-off-by: Balazs Kutil <kutil.balazs@gmail.com>
    Balazs Kutil committed Nov 4, 2011
    Configuration menu
    Copy the full SHA
    ff721cd View commit details
    Browse the repository at this point in the history