Skip to content

Commit a8bbdf8

Browse files
addie9800bucheben
authored andcommitted
remove unnecessary try/except
1 parent d44dd65 commit a8bbdf8

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

src/fundus/scraping/url.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
import feedparser
1212
import lxml.html
1313
import validators
14-
from lxml.etree import XMLParser, XMLSyntaxError, XPath
14+
from lxml.etree import XMLParser, XPath
1515
from requests import ConnectionError, HTTPError, ReadTimeout
1616

1717
from fundus.logging import create_logger
@@ -194,11 +194,7 @@ def yield_recursive(sitemap_url: str) -> Iterator[str]:
194194
if not content:
195195
logger.warning(f"Warning! Empty sitemap at {sitemap_url!r}")
196196
return
197-
try:
198-
tree = lxml.etree.fromstring(content, parser=self._parser)
199-
except XMLSyntaxError:
200-
logger.warning(f"Warning! Couldn't parse sitemap {sitemap_url!r} because of invalid XML")
201-
return
197+
tree = lxml.etree.fromstring(content, parser=self._parser)
202198
urls = [node.text for node in self._url_selector(tree)]
203199
if urls:
204200
for new_url in reversed(urls) if self.reverse else urls:

0 commit comments

Comments
 (0)