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

HTTPS does not handle pipelined requests #37008

Closed
jhylton mannequin opened this issue Aug 8, 2002 · 8 comments
Closed

HTTPS does not handle pipelined requests #37008

jhylton mannequin opened this issue Aug 8, 2002 · 8 comments
Labels
stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error

Comments

@jhylton
Copy link
Mannequin

jhylton mannequin commented Aug 8, 2002

BPO 592703
Nosy @birkenfeld, @amauryfa, @orsenthil, @pitrou

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2013-05-26.19:55:57.219>
created_at = <Date 2002-08-08.17:03:20.000>
labels = ['type-bug', 'library']
title = 'HTTPS does not handle pipelined requests'
updated_at = <Date 2013-05-26.19:55:57.217>
user = 'https://bugs.python.org/jhylton'

bugs.python.org fields:

activity = <Date 2013-05-26.19:55:57.217>
actor = 'pitrou'
assignee = 'janssen'
closed = True
closed_date = <Date 2013-05-26.19:55:57.219>
closer = 'pitrou'
components = ['Library (Lib)']
creation = <Date 2002-08-08.17:03:20.000>
creator = 'jhylton'
dependencies = []
files = []
hgrepos = []
issue_num = 592703
keywords = []
message_count = 8.0
messages = ['60273', '61282', '61315', '114181', '114260', '189986', '190107', '190110']
nosy_count = 7.0
nosy_names = ['jhylton', 'georg.brandl', 'amaury.forgeotdarc', 'janssen', 'orsenthil', 'pitrou', 'BreamoreBoy']
pr_nums = []
priority = 'normal'
resolution = 'works for me'
stage = 'test needed'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue592703'
versions = ['3rd party']

@jhylton
Copy link
Mannequin Author

jhylton mannequin commented Aug 8, 2002

No test case yet, but the buffering code for SSLFile
looks like it will break pipelined requests.

@jhylton jhylton mannequin self-assigned this Aug 8, 2002
@jhylton jhylton mannequin added the stdlib Python modules in the Lib dir label Aug 8, 2002
@jhylton jhylton mannequin self-assigned this Aug 8, 2002
@jhylton jhylton mannequin added the stdlib Python modules in the Lib dir label Aug 8, 2002
@birkenfeld
Copy link
Member

SSL support was rewritten, so this may be fixed now...

@birkenfeld birkenfeld assigned janssen and unassigned jhylton Jan 20, 2008
@janssen
Copy link
Mannequin

janssen mannequin commented Jan 20, 2008

Well, if someone writes test case and verifies it, I'll look at it.

@devdanzin devdanzin mannequin added type-bug An unexpected behavior, bug, or error labels Feb 13, 2009
@BreamoreBoy
Copy link
Mannequin

BreamoreBoy mannequin commented Aug 17, 2010

is this still an issue or can it be closed as out fo date?

@orsenthil
Copy link
Member

This needs investigation. Don't close.

@BreamoreBoy
Copy link
Mannequin

BreamoreBoy mannequin commented May 25, 2013

Bump.

@amauryfa
Copy link
Member

Tried with python2.7.3
I used http://code.activestate.com/recipes/576673-python-http-pipelining/ just replacing the initial call to HTTPConnection() by HTTPSConnection().

The example succeeeds, fetches the three pages, and I checked with strace that the same connection is used for all requests (only reads and writes of encrypted content, no close or connect in between).

Is this enough for closing this issue? Or is there a different way to pipelines requests that we should support?

@pitrou
Copy link
Member

pitrou commented May 26, 2013

The original report is so under-specified that it's pointless to keep it open. If you encounter an actual bug, feel free to open another issue :)

@pitrou pitrou closed this as completed May 26, 2013
@pitrou pitrou closed this as completed May 26, 2013
@ezio-melotti ezio-melotti transferred this issue from another repository Apr 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

No branches or pull requests

4 participants