Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Normalize redirects

#25

in the http.rb:

redirect_to = response.is_a?(Net::HTTPRedirection) ?  URI(response['location']): nil
should be
redirect_to = response.is_a?(Net::HTTPRedirection) ?  URI(response['location']).normalize : nil
(added .normalize)
Otherwise we run crash because we run into redirects like "http://www.example.org"
The current crawler doesn't allow root level redirects, sanitizing input would still would be the correct thing to do though:)

Reported by rb2k · May 18th, 2010 @ 07:46 AM

State: resolved
Milestone: 0.4.1
Assigned to: chris (at chriskite) chris (at chriskite)

Activity

  1. chris (at chriskite)
    chris (at chriskite)
    • Milestone set to 0.4.1
    • State changed from new to resolved

    May 25th, 2010 @ 09:06 PM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.