From: Christian Heller Date: Mon, 18 Jan 2016 21:25:14 +0000 (+0100) Subject: Be verbose on page title retrieval failure. X-Git-Url: https://plomlompom.com/repos/%7B%7Bdb.prefix%7D%7D/%7B%7B%20web_path%20%7D%7D/decks/tasks?a=commitdiff_plain;h=3b3ac1e18357e60525a6be9b5665c504e0ec4e6c;p=plomlombot-irc.git Be verbose on page title retrieval failure. --- diff --git a/plomlombot.py b/plomlombot.py index 7d0f535..81be31f 100755 --- a/plomlombot.py +++ b/plomlombot.py @@ -100,6 +100,10 @@ def lineparser_loop(io, nickname): def act_on_privmsg(tokens): def url_check(msg): + + def notice(msg): + io.send_line("NOTICE " + target + " :" + msg) + matches = re.findall("(https?://[^\s>]+)", msg) for i in range(len(matches)): url = matches[i] @@ -109,9 +113,8 @@ def lineparser_loop(io, nickname): try: webpage = urllib.request.urlopen(request, timeout=15) except (urllib.error.HTTPError, urllib.error.URLError, - UnicodeError, http.client.BadStatusLine, - UnicodeDecodeError) as error: - print("TROUBLE FOLLOWING URL: " + str(error)) + UnicodeError, http.client.BadStatusLine) as error: + notice("TROUBLE FOLLOWING URL: " + str(error)) continue charset = webpage.info().get_content_charset() if not charset: @@ -119,14 +122,13 @@ def lineparser_loop(io, nickname): content_type = webpage.info().get_content_type() if content_type not in ('text/html', 'text/xml', 'application/xhtml+xml'): - print("TROUBLE INTERPRETING URL: bad content type " - + content_type) + notice("TROUBLE READING PAGE TITLE: bad content type " + + content_type) continue content = webpage.read().decode(charset) title = str(content).split('')[1].split('')[0] title = html.unescape(title) - io.send_line("NOTICE " + target + " :page title for url: " - + title) + notice("PAGE TITLE FOR URL: " + title) sender = "" for rune in tokens[0]: