Bug 906924

Summary: [patch] Backport support correct cookie handling
Product: [Fedora] Fedora Reporter: Ralph Bean <rbean>
Component: python-requestsAssignee: Arun S A G <sagarun>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 18CC: a.badger, ppisar, robatino, sagarun, samuel
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-02-16 22:32:05 EST Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Description Flags
Handle cookies correctly
Changes to the spec file
Patch to enable the system cert bundle none

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.