Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle ConnectionError on HTTP HEAD Requests #9308

Closed
wants to merge 199 commits into from

Conversation

jamathews
Copy link
Contributor

Subject: Resolves #9306

Feature or Bugfix

  • Bugfix

Purpose

Detail

  • The US Patent and Trademark website seems to close connections on HTTP HEAD requests, but will return the requested document for HTTP GET requests. The linkcheck.py code wasn't catching the ConnectionError exception, and as a result the code would skip the requests.get() attempt. With this fix, if requests.head() raises a ConnectionError, it will attempt a requests.get() which, for the UT Patent website, succeeds. If the website truly is rejecting all connections, an exception on the requests.get() call will still send control to the except Exception as err block as before.

Relates

tk0miya and others added 30 commits April 11, 2021 22:53
…rammar-fix

doc: Fix grammar in documentation for `autodoc_typehints`
Remove unnecesary sleep when submitting parallel tasks
Recommend MyST-Parser for Markdown usage
* Remove `div.doctest` so that the selecting feature in snippets code
are not just available for doctests snippets
C++, add internal API for setting name in ASTType
This seems to have been a mistake with sphinx-doc#4022 the ajax call functions correctly without the source files being included in the build (they are never used).

I have tested this out on several themes and now everything works correctly with `html_copy_source = False`
There is no docs and testcases for "None" argument of the program
directive.  It has been implemented since very old version.  But it's
not documented and tested long.
Since checkmark already exists, it makes sense to have this too.
Both signs are used as "yes" or "no" replacements in text UIs.
…ultiplication

Add unicode multiplication to texescape
Add doc and testcase for program directive (refs: sphinx-doc#9137)
tk0miya and others added 25 commits June 3, 2021 21:47
…e_not_having_comment

Fix sphinx-doc#9283: autodoc: failed to build doc for attribute not commented
Co-authored-by: François Freitag <mail@franek.fr>
Except for py which has mixins that assumes a single pending_xref
The err role doesn't exist and the rest are all equivalent.
…_anchors

Close sphinx-doc#9016: linkcheck builder failed to check the anchors of github.com
fix: Fix typo in add_directive docstring
doc: Fix docslint misunderstanding scope of code-blocks
@jamathews
Copy link
Contributor Author

Wrong branch, opened #9309 to replace this.

@jamathews jamathews closed this Jun 7, 2021
Copy link

@elmayana00 elmayana00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Want to try this out

@jamathews
Copy link
Contributor Author

Want to try this out

@elmayana00 I messed up this PR by selecting branches 4.x to 4.0.x. #9309 is the clean PR that's 4.0.x to 4.0.x.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.