[issue27996] Python 3 ssl module can't use a fileno to create a SSLSocket

2016-09-07 Thread Berker Peksag

Berker Peksag added the comment:

Great, closing this then.

> And the next release will include this patch?

Christian (one of our SSL module maintainers) just reviewed the patch so the 
next step is to address his review comments and upload a new patch. If you 
don't see any movement from nemunaire (the original author of the patch) after 
a week or so, you can update the patch yourself (it would help to get it fixed 
sooner.)

--
resolution:  -> duplicate
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue27996] Python 3 ssl module can't use a fileno to create a SSLSocket

2016-09-07 Thread SenBin Yu

SenBin Yu added the comment:

Yes, the giving code is what i desiere and shoud be supported. Sorry, i haven't 
see the patch file.And the next release will include this patch?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue27996] Python 3 ssl module can't use a fileno to create a SSLSocket

2016-09-07 Thread Berker Peksag

Berker Peksag added the comment:

Did you look at the patch? It adds the following test case:

+s = socket.socket()
+sfd = s.fileno()
+# Create secure socket from fileno
+ss = ssl.SSLSocket(fileno=sfd)
+self.assertTrue(ss.fileno() == sfd)

Isn't that what you want to achieve?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue27996] Python 3 ssl module can't use a fileno to create a SSLSocket

2016-09-07 Thread SenBin Yu

SenBin Yu added the comment:

The issue 27629 describe creating a SSLSocket only with server_hostname 
argument.But what i said is that we can't creating a SSLSocket with fileno 
argument from a existing socket.I'm strange about this code design that 
meaningless branch conditions exists.

--
resolution: duplicate -> 
status: closed -> open

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue27996] Python 3 ssl module can't use a fileno to create a SSLSocket

2016-09-07 Thread Berker Peksag

Berker Peksag added the comment:

Thanks for the report. This looks like a duplicate of issue 27629. Please 
provide a reproducer if it's different problem than issue 27629.

--
nosy: +berker.peksag
resolution:  -> duplicate
stage:  -> resolved
status: open -> closed
superseder:  -> Cannot create ssl.SSLSocket without existing socket

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue27996] Python 3 ssl module can't use a fileno to create a SSLSocket

2016-09-06 Thread SenBin Yu

New submission from SenBin Yu:

In the SSLSocket __init__ function, it's ok to do a socket type check with 
argument sock.But meanwhile the sock argument can't be None, which make the 
SSLSocket socket __init__ way meaningless.Although there are three branch 
conditions as argument sock, fileno and other.

--
components: Library (Lib)
files: SSLSocket_sock.png
messages: 274740
nosy: SenBin Yu
priority: normal
severity: normal
status: open
title: Python 3 ssl module can't use a fileno to create a SSLSocket
type: behavior
versions: Python 3.5
Added file: http://bugs.python.org/file44423/SSLSocket_sock.png

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com