Bug 906924 - [patch] Backport support correct cookie handling
[patch] Backport support correct cookie handling
Product: Fedora
Classification: Fedora
Component: python-requests (Show other bugs)
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: Arun S A G
Fedora Extras Quality Assurance
: 909296 909444 (view as bug list)
Depends On:
  Show dependency treegraph
Reported: 2013-02-01 17:31 EST by Ralph Bean
Modified: 2013-02-16 22:32 EST (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2013-02-16 22:32:05 EST
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
Handle cookies correctly (1.33 KB, patch)
2013-02-01 17:31 EST, Ralph Bean
no flags Details | Diff
Changes to the spec file (4.73 KB, patch)
2013-02-08 15:25 EST, Toshio Ernie Kuratomi
no flags Details | Diff
Patch to enable the system cert bundle (523 bytes, patch)
2013-02-08 15:25 EST, Toshio Ernie Kuratomi
no flags Details | Diff

  None (edit)
Description Ralph Bean 2013-02-01 17:31:47 EST
Created attachment 691772 [details]
Handle cookies correctly

We found a weird corner case in python-requests-0.14.1 in the way that it handles cookies.  It is discussed here:  https://github.com/fedora-infra/python-fedora/pull/6

Fixing it ended up requiring a patch that I have added to python-requests-0.14.1-3.el6.  This is a request to merge that patch back into rawhide and f18.

The patch is attached.  A simple "git merge" from el6 into the master and f18 branches should do it.
Comment 1 Toshio Ernie Kuratomi 2013-02-01 17:37:16 EST
One note -- the patch is not quite a backport.  The python-requests-1.0.0 release doesn't have this problem but 1.0.0 was a huge restructuring of code.  So this was fix doesn't have an analogoue in the upstream 1.0.0 tree.
Comment 2 Toshio Ernie Kuratomi 2013-02-08 10:13:15 EST
*** Bug 909296 has been marked as a duplicate of this bug. ***
Comment 3 Toshio Ernie Kuratomi 2013-02-08 10:15:21 EST
Arun, mind if I apply the patch and rebuild?  The broken behavior was noticed by someone running fedora-easy-karma today.
Comment 4 Arun S A G 2013-02-08 11:44:48 EST
Sure,  Not a problem. Go ahead!
Comment 5 Toshio Ernie Kuratomi 2013-02-08 15:21:31 EST
I got carried away a little bit.  Made a few changes above and beyond what we talked about here.  I'll upload the patch and then build it if either one of you okay it.
Comment 6 Toshio Ernie Kuratomi 2013-02-08 15:25:16 EST
Created attachment 695229 [details]
Changes to the spec file

This fixes the cookie handling, fixes the certificate bundle in https://bugzilla.redhat.com/show_bug.cgi?id=904614 , and removes the py-3 specific and py2-specific modules in the packages where they won't run (chardet is not in python-requests and chardet2, oauthlib is removed from python3-requests) so that we can use the generic fedora python byte compilation.
Comment 7 Toshio Ernie Kuratomi 2013-02-08 15:25:52 EST
Created attachment 695230 [details]
Patch to enable the system cert bundle
Comment 8 Ralph Bean 2013-02-08 15:31:18 EST
+1 to the patches.  Thanks for all your work on this Toshio!
Comment 9 Fedora Update System 2013-02-08 17:31:15 EST
python-fedora-, python-requests-0.14.1-4.fc18 has been submitted as an update for Fedora 18.
Comment 10 Toshio Ernie Kuratomi 2013-02-09 11:19:02 EST
*** Bug 909444 has been marked as a duplicate of this bug. ***
Comment 11 Fedora Update System 2013-02-09 23:43:16 EST
Package python-fedora-, python-requests-0.14.1-4.fc18:
* should fix your issue,
* was pushed to the Fedora 18 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing python-fedora- python-requests-0.14.1-4.fc18'
as soon as you are able to.
Please go to the following url:
then log in and leave karma (feedback).
Comment 12 Fedora Update System 2013-02-16 22:32:07 EST
python-fedora-, python-requests-0.14.1-4.fc18 has been pushed to the Fedora 18 stable repository.  If problems still persist, please make note of it in this bug report.

Note You need to log in before you can comment on or make changes to this bug.