From 1ae131c231c308f13eaed68a3564c19f7eb7f2f1 Mon Sep 17 00:00:00 2001 From: Giuseppe Bilotta Date: Mon, 10 Mar 2008 23:09:09 +0100 Subject: url plugin: retry with chops even on URI::InvalidURIError --- data/rbot/plugins/url.rb | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) (limited to 'data/rbot/plugins/url.rb') diff --git a/data/rbot/plugins/url.rb b/data/rbot/plugins/url.rb index e1c9b473..b048d3f0 100644 --- a/data/rbot/plugins/url.rb +++ b/data/rbot/plugins/url.rb @@ -140,9 +140,13 @@ class UrlPlugin < Plugin debug "Title #{title ? '' : 'not '} found" reply = "#{LINK_INFO} #{title}" if title rescue => e - if e.message =~ /\(404 - Not Found\)/i - # see if we failed to find the thing because of trailing punctuation - # but check that we still have 'something' in the URL + debug e + # we might get a 404 because of trailing punctuation, so we try again + # with the last character stripped. this might generate invalid URIs + # (e.g. because "some.url" gets chopped to some.url%2, so catch that too + if e.message =~ /\(404 - Not Found\)/i or e.kind_of?(URI::InvalidURIError) + # chop off last character, and retry if we still have enough string to + # look like a minimal URL retry if urlstr.chop! and urlstr =~ /^https?:\/\/./ end reply = "Error #{e.message}" -- cgit v1.2.3