shorter explanatory comments

Co-authored-by: François Freitag <mail@franek.fr>
This commit is contained in:
Justin Mathews 2021-06-10 11:40:44 -04:00 committed by GitHub
parent d804981a37
commit ce305190c5
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -460,11 +460,8 @@ class HyperlinkAvailabilityCheckWorker(Thread):
config=self.config, auth=auth_info,
**kwargs)
response.raise_for_status()
# When there is a ConnectionError from the HEAD request, it might be due to
# the webserver intentionally dropping the connection when it sees HEAD requests
# but still responding properly to GET requests. This has been observed in the
# wild with at https://patft.uspto.gov/.
# See https://github.com/sphinx-doc/sphinx/issues/9306 for more details.
# Servers drop the connection on HEAD requests, causing
# ConnectionError.
except (ConnectionError, HTTPError, TooManyRedirects) as err:
if isinstance(err, HTTPError) and err.response.status_code == 429:
raise