Try to detect when users need to Install Certificates on Mac (realpython#16)

gahjelle · web-flow · commit bbc8e8f39c47 · 2023-03-29T11:13:34.000+02:00
* Point users to Install Certificates on Mac

* Use error message to detect certificate issue

* Remove outdated noqa comment
diff --git a/src/reader/feed.py b/src/reader/feed.py
@@ -1,6 +1,6 @@
 """Interact with the Real Python feed."""
 # Standard library imports
-from typing import Dict, List  # noqa
+from typing import Dict, List
 
 # Third party imports
 import feedparser
@@ -21,8 +21,17 @@ def _feed(url: str = URL) -> feedparser.FeedParserDict:
 
 def get_site(url: str = URL) -> str:
     """Get name and link to website of the feed."""
-    info = _feed(url).feed
-    return f"{info.title} ({info.link})"
+    info = _feed(url)
+    if exception := info.get("bozo_exception"):
+        message = f"Could not read feed at {url}"
+        if "CERTIFICATE_VERIFY_FAILED" in str(exception):
+            message += (
+                ".\n\nYou may need to manually install certificates by running "
+                "`Install Certificates` in your Python installation folder. "
+                "See https://realpython.com/installing-python/"
+            )
+        raise SystemExit(message)
+    return f"{info.feed.title} ({info.feed.link})"
 
 
 def get_article(article_id: str, links: bool = False, url: str = URL) -> str:
