Commits on May 17, 2023

1. fix: HTTP 308 Permanent Redirect status code handling …

   Change the handling of HTTP status code 308 to behave more like
   `urllib.request.HTTPRedirectHandler`, most critically, the new 308 handling will
   create a new `urllib.request.Request` object with the new URL, which will
   prevent state from being carried over from the original request.
   
   One case where this is important is when the domain name changes, for example,
   when the original URL is `http://www.w3.org/ns/adms.ttl` and the redirect URL is
   `https://uri.semic.eu/w3c/ns/adms.ttl`. With the previous behaviour, the redirect
   would contain a `Host` header with the value `www.w3.org` instead of
   `uri.semic.eu`, because the `Host` header is placed in
   `Request.unredirected_hdrs` and takes precedence over the `Host` header in
   `Request.headers`.
   
   Other changes:
   - Only handle HTTP status code 308 on Python versions before 3.11 as Python 3.11
     will handle 308 by default [[ref](https://docs.python.org/3.11/whatsnew/changelog.html#id128)].
   - Move code which uses `http://www.w3.org/ns/adms.ttl` and
     `http://www.w3.org/ns/adms.rdf` out of `test_guess_format_for_parse` into a
     separate parameterized test which instead uses the embedded http server.
   
     This allows the test to fully control the `Content-Type` header in the
     response instead of relying on the value that the server is sending.
   
     This is needed because the server is sending `Content-Type: text/plain` for
     the `adms.ttl` file, which is not a valid RDF format, and the test is
     expecting `Content-Type: text/turtle`.
   
   Fixes:
   - <RDFLib#2382>.

   

   aucampia committed May 17, 2023
   Configuration menu

   • View commit details
   • Copy full SHA for 90071df
   • Browse repository at this point

   Copy the full SHA

   90071df View commit details

   Browse the repository at this point in the history