Use the new encapsulated fetchers

[paperdoorknob] / paperdoorknob.py

diff --git a/paperdoorknob.py b/paperdoorknob.py
index 8ab62f10b655f6b9f7c21cc4dd5a1f5379d066be..f83c9f869651a0a35fd86fdf11fdb004e5e5ae4d 100644 (file)
--- a/paperdoorknob.py
+++ b/paperdoorknob.py
@@ -14,10 +14,10 @@ from typing import IO, Iterable
 
 from bs4 import BeautifulSoup
 from bs4.element import Tag
 
 from bs4 import BeautifulSoup
 from bs4.element import Tag

-import requests
-import requests_cache

 from xdg_base_dirs import xdg_cache_home
 
 from xdg_base_dirs import xdg_cache_home
 

+from fetch import CachingFetcher, Fetcher
+

 
 def command_line_parser() -> ArgumentParser:
     parser = ArgumentParser(prog='paperdoorknob', description='Print glowfic')
 
 def command_line_parser() -> ArgumentParser:
     parser = ArgumentParser(prog='paperdoorknob', description='Print glowfic')


@@ -40,10 +40,8 @@ def command_line_parser() -> ArgumentParser:
     return parser
 
 
     return parser
 
 

-def fetch(url: str, session: requests.Session, timeout: int) -> BeautifulSoup:
-    with session.get(url, timeout=timeout) as r:
-        r.raise_for_status()
-        return BeautifulSoup(r.text, 'html.parser')
+def fetch(url: str, fetcher: Fetcher) -> BeautifulSoup:
+    return BeautifulSoup(fetcher.fetch(url), 'html.parser')

 
 
 def clean(html: BeautifulSoup) -> BeautifulSoup:
 
 
 def clean(html: BeautifulSoup) -> BeautifulSoup:


@@ -79,13 +77,11 @@ def html_to_tex(pandoc: str, tag: Tag) -> bytes:
 
 def process(
         url: str,
 
 def process(
         url: str,

-        session: requests.Session,
-        timeout: int,
+        fetcher: Fetcher,

         texout: IO[bytes],
         texout: IO[bytes],

-        pandoc: str,
-) -> None:
+        pandoc: str) -> None:

     texout.write(b'\\documentclass{article}\n\\begin{document}\n')
     texout.write(b'\\documentclass{article}\n\\begin{document}\n')

-    html = clean(fetch(url, session, timeout))
+    html = clean(fetch(url, fetcher))

     for r in replies(html):
         texout.write(html_to_tex(pandoc, r))
     texout.write(b'\\end{document}\n')
     for r in replies(html):
         texout.write(html_to_tex(pandoc, r))
     texout.write(b'\\end{document}\n')


@@ -93,14 +89,9 @@ def process(
 
 def main() -> None:
     args = command_line_parser().parse_args()
 
 def main() -> None:
     args = command_line_parser().parse_args()

-    with requests_cache.CachedSession(args.cache_path, cache_control=True) as session:
+    with CachingFetcher(args.cache_path, args.timeout) as fetcher:

         with open(args.out + '.tex', 'wb') as texout:
         with open(args.out + '.tex', 'wb') as texout:

-            process(
-                args.url,
-                session,
-                args.timeout,
-                texout,
-                args.pandoc or 'pandoc')
+            process(args.url, fetcher, texout, args.pandoc or 'pandoc')

 
 
 if __name__ == '__main__':
 
 
 if __name__ == '__main__':