Hello community,
here is the log from the commit of package python-tornado6 for openSUSE:Factory
checked in at 2020-05-08 23:00:40
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-tornado6 (Old)
and /work/SRC/openSUSE:Factory/.python-tornado6.new.2738 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "python-tornado6"
Fri May 8 23:00:40 2020 rev:4 rq:800447 version:6.0.4
Changes:
--------
--- /work/SRC/openSUSE:Factory/python-tornado6/python-tornado6.changes
2020-03-27 00:19:55.680094117 +0100
+++
/work/SRC/openSUSE:Factory/.python-tornado6.new.2738/python-tornado6.changes
2020-05-08 23:00:41.493291057 +0200
@@ -1,0 +2,8 @@
+Tue May 5 16:36:45 UTC 2020 - Pedro Monreal Gonzalez
<[email protected]>
+
+- Fix build with curl 7.70.0:
+ * Revert commit c443fb7bf8a87ba8ab02b9a6af9e140cabc0ab0d which
+ introduces test_method_after_redirect() test.
+- Add python-tornado6-httpclient-test.patch
+
+-------------------------------------------------------------------
New:
----
python-tornado6-httpclient-test.patch
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ python-tornado6.spec ++++++
--- /var/tmp/diff_new_pack.2UIvRK/_old 2020-05-08 23:00:42.105292307 +0200
+++ /var/tmp/diff_new_pack.2UIvRK/_new 2020-05-08 23:00:42.109292315 +0200
@@ -23,11 +23,12 @@
Release: 0
Summary: Open source version of scalable, non-blocking web server that
power FriendFeed
License: Apache-2.0
-URL: http://www.tornadoweb.org
+URL: https://www.tornadoweb.org
Source:
https://files.pythonhosted.org/packages/source/t/tornado/tornado-%{version}.tar.gz
Patch1: tornado-testsuite_timeout.patch
Patch2: skip-failing-tests.patch
Patch3: ignore-resourcewarning-doctests.patch
+Patch4: python-tornado6-httpclient-test.patch
BuildRequires: %{python_module base >= 3.5}
BuildRequires: %{python_module certifi}
BuildRequires: %{python_module devel}
++++++ python-tornado6-httpclient-test.patch ++++++
diff -PpuriN tornado-6.0.4/tornado/simple_httpclient.py
tornado-6.0.4-mod/tornado/simple_httpclient.py
--- tornado-6.0.4/tornado/simple_httpclient.py 2020-03-02 20:21:37.000000000
+0100
+++ tornado-6.0.4-mod/tornado/simple_httpclient.py 2020-05-05
17:52:46.375282118 +0200
@@ -626,17 +626,14 @@ class _HTTPConnection(httputil.HTTPMessa
)
new_request.max_redirects = self.request.max_redirects - 1
del new_request.headers["Host"]
- # https://tools.ietf.org/html/rfc7231#section-6.4
- #
- # The original HTTP spec said that after a 301 or 302
- # redirect, the request method should be preserved.
- # However, browsers implemented this by changing the
- # method to GET, and the behavior stuck. 303 redirects
- # always specified this POST-to-GET behavior (arguably 303
- # redirects should change *all* requests to GET, but
- # libcurl only does this for POST so we follow their
- # example).
- if self.code in (301, 302, 303) and self.request.method == "POST":
+ # http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html#sec10.3.4
+ # Client SHOULD make a GET request after a 303.
+ # According to the spec, 302 should be followed by the same
+ # method as the original request, but in practice browsers
+ # treat 302 the same as 303, and many servers use 302 for
+ # compatibility with pre-HTTP/1.1 user agents which don't
+ # understand the 303 status.
+ if self.code in (302, 303):
new_request.method = "GET"
new_request.body = None
for h in [
diff -PpuriN tornado-6.0.4/tornado/test/httpclient_test.py
tornado-6.0.4-mod/tornado/test/httpclient_test.py
--- tornado-6.0.4/tornado/test/httpclient_test.py 2020-03-02
20:21:37.000000000 +0100
+++ tornado-6.0.4-mod/tornado/test/httpclient_test.py 2020-05-05
17:53:39.259950459 +0200
@@ -126,7 +126,7 @@ class AllMethodsHandler(RequestHandler):
def method(self):
self.write(self.request.method)
- get = head = post = put = delete = options = patch = other = method #
type: ignore
+ get = post = put = delete = options = patch = other = method # type:
ignore
class SetHeaderHandler(RequestHandler):
@@ -322,32 +322,6 @@ Transfer-Encoding: chunked
)
self.assertEqual(response.body, b"Put body: ")
- def test_method_after_redirect(self):
- # Legacy redirect codes (301, 302) convert POST requests to GET.
- for status in [301, 302, 303]:
- url = "/redirect?url=/all_methods&status=%d" % status
- resp = self.fetch(url, method="POST", body=b"")
- self.assertEqual(b"GET", resp.body)
-
- # Other methods are left alone.
- for method in ["GET", "OPTIONS", "PUT", "DELETE"]:
- resp = self.fetch(url, method=method,
allow_nonstandard_methods=True)
- self.assertEqual(utf8(method), resp.body)
- # HEAD is different so check it separately.
- resp = self.fetch(url, method="HEAD")
- self.assertEqual(200, resp.code)
- self.assertEqual(b"", resp.body)
-
- # Newer redirects always preserve the original method.
- for status in [307, 308]:
- url = "/redirect?url=/all_methods&status=307"
- for method in ["GET", "OPTIONS", "POST", "PUT", "DELETE"]:
- resp = self.fetch(url, method=method,
allow_nonstandard_methods=True)
- self.assertEqual(method, to_unicode(resp.body))
- resp = self.fetch(url, method="HEAD")
- self.assertEqual(200, resp.code)
- self.assertEqual(b"", resp.body)
-
def test_credentials_in_url(self):
url = self.get_url("/auth").replace("http://";, "http://me:secret@";)
response = self.fetch(url)
