X-Git-Url: https://git.phdru.name/?a=blobdiff_plain;f=Robots%2Fbkmk_rsimple.py;h=77c7a150109509583c2eb3d5fb5f538d45806283;hb=d12f0f2bd47efb4c41dcd306c53d712ed3d194d7;hp=af549f63969001f87e725b90df87811552512491;hpb=c067541ef9e42e33742ad90e1a51c9aea2bcdc51;p=bookmarks_db.git

diff --git a/Robots/bkmk_rsimple.py b/Robots/bkmk_rsimple.py
index af549f6..77c7a15 100644
--- a/Robots/bkmk_rsimple.py
+++ b/Robots/bkmk_rsimple.py
@@ -44,8 +44,7 @@ class MyURLopener(urllib.URLopener):
 
 urllib._urlopener = MyURLopener()
 
-# Some sites allow only Mozilla-compatible browsers; way to stop robots?
-server_version = "Mozilla/3.0 (compatible; Python-urllib/%s)" % urllib.__version__
+server_version = "bookmarks_db (Python-urllib/%s)" % urllib.__version__
 urllib._urlopener.addheaders[0] = ('User-agent', server_version)
 
 
@@ -83,13 +82,16 @@ from bkmk_objects import Robot
 from parse_html import parse_html
 
 class robot_simple(Robot):
-   def check_url(self, bookmark, url_type, url_rest):
+   def check_url(self, bookmark):
       if not self.tempfname:
          self.tempfname = bookmark.tempfname
 
       try:
          try:
             self.start = int(time.time())
+            bookmark.icon = None
+
+            url_type, url_rest = urllib.splittype(bookmark.href)
             url_host, url_path = urllib.splithost(url_rest)
             url_path, url_tag  = urllib.splittag(url_path)
 
@@ -142,9 +144,7 @@ class robot_simple(Robot):
                      if self.log: self.log("   no charset in Content-Type header")
                   if content_type == "text/html":
                      parser = parse_html(fname, charset, self.log)
-                     title = parser.title.replace('\r', '').replace('\n', ' ').strip()
-                     bookmark.real_title = parser.unescape(title)
-                     if self.log: self.log("   final title    : %s" % bookmark.real_title)
+                     bookmark.real_title = parser.title
                      if parser.refresh:
                         refresh = parser.refresh
                         try:
@@ -152,19 +152,17 @@ class robot_simple(Robot):
                         except IndexError:
                            url = "self"
                         try:
-                           timeout = int(refresh.split(';')[0])
+                           timeout = float(refresh.split(';')[0])
                         except (IndexError, ValueError):
-                           timeout = None
-                        if timeout is None:
                            raise RedirectException("html", "Bad redirect to %s (%s)" % (url, refresh))
                         else:
-                           raise RedirectException("html", "%s (%d sec)" % (url, timeout))
+                           raise RedirectException("html", "%s (%s sec)" % (url, timeout))
 
                      # Get favicon.ico
                      icon = parser.icon
                      if not icon:
                         icon = "/favicon.ico"
-                     icon = urljoin("%s://%s" % (url_type, url_host), icon)
+                     icon = urljoin("%s://%s%s" % (url_type, url_host, url_path), icon)
                      if self.log: self.log("   icon           : %s" % icon)
                      try:
                         fname, headers = urllib.urlretrieve(icon)
@@ -175,12 +173,15 @@ class robot_simple(Robot):
                         emsg = None
                         tb = None
                      else:
-                        icon_file = open(fname, "rb")
-                        icon = icon_file.read()
-                        icon_file.close()
-                        os.remove(fname)
                         content_type = headers["Content-Type"]
-                        bookmark.icon = "data:%s;base64,%s" % (content_type, b64encode(icon))
+                        if content_type.startswith("image/"):
+                           icon_file = open(fname, "rb")
+                           icon = icon_file.read()
+                           icon_file.close()
+                           bookmark.icon = "data:%s;base64,%s" % (content_type, b64encode(icon))
+                        else:
+                           if self.log: self.log("   no icon        : %s" % content_type)
+                        os.remove(fname)
 
                except KeyError:
                   pass