[issue1489] test_socket_ssl hanhs on Windows (deadlock)

[Amaury Forgeot d'Arc]

Amaury Forgeot d'Arc <[EMAIL PROTECTED]> added the comment:

Kind of fixed: test_socket_ssl was removed in both trunk and py3k branches.

"tests are now in test_ssl.py"

--
nosy: +amaury.forgeotdarc
resolution:  -> fixed
status: open -> closed

___
Python tracker <[EMAIL PROTECTED]>

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

[issue1489] test_socket_ssl hanhs on Windows (deadlock)

[Bill Janssen]

Bill Janssen <[EMAIL PROTECTED]> added the comment:

Sorry, didn't mean to be indeterminate.  Sure, nuke it.  The tests are now
all in test_ssl.py.

Bill

On Tue, Apr 22, 2008 at 2:47 PM, Bill Janssen <[EMAIL PROTECTED]>
wrote:

>
>
> On Tue, Apr 22, 2008 at 12:15 PM, Trent Nelson <[EMAIL PROTECTED]>
> wrote:
>
> >
> > Trent Nelson <[EMAIL PROTECTED]> added the comment:
> >
> > Hey, if we can get rid of it, that's great.  You sound 95% certain in
> > your last post that completely removing test_socket_ssl on trunk and
> > py3k is the Right Thing -- any chance of getting you to 100% so I can
> > go ahead and delete it and close this ticket?
> >
> > __
> > Tracker <[EMAIL PROTECTED]>
> > 
> > __
> >
>
>

Added file: http://bugs.python.org/file10075/unnamed

__
Tracker <[EMAIL PROTECTED]>

__Sorry, didn't mean to be indeterminate.  Sure, nuke it.  The 
tests are now all in test_ssl.py.BillOn Tue, Apr 22, 2008 at 2:47 PM, Bill Janssen [EMAIL PROTECTED]> wrote:
On Tue, Apr 22, 2008 at 12:15 
PM, Trent Nelson [EMAIL 
PROTECTED]> wrote:


Trent Nelson [EMAIL 
PROTECTED]> added the comment:

Hey, if we can get rid of it, that's great.  You sound 95% 
certain in
your last post that completely removing test_socket_ssl on trunk and
py3k is the Right Thing -- any chance of getting you to 100% so I can
go ahead and delete it and close this ticket?

__
Tracker [EMAIL 
PROTECTED]>
http://bugs.python.org/issue1489>
__


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

[issue1489] test_socket_ssl hanhs on Windows (deadlock)

[Bill Janssen]

Bill Janssen <[EMAIL PROTECTED]> added the comment:

On Tue, Apr 22, 2008 at 12:15 PM, Trent Nelson <[EMAIL PROTECTED]>
wrote:

>
> Trent Nelson <[EMAIL PROTECTED]> added the comment:
>
> Hey, if we can get rid of it, that's great.  You sound 95% certain in
> your last post that completely removing test_socket_ssl on trunk and
> py3k is the Right Thing -- any chance of getting you to 100% so I can
> go ahead and delete it and close this ticket?
>
> __
> Tracker <[EMAIL PROTECTED]>
> 
> __
>

--
title: test_socket_ssl hangs on Windows (deadlock) -> test_socket_ssl hanhs on 
Windows (deadlock)
Added file: http://bugs.python.org/file10074/unnamed

__
Tracker <[EMAIL PROTECTED]>

__On Tue, Apr 22, 2008 at 12:15 PM, Trent Nelson 
[EMAIL PROTECTED]> 
wrote:

Trent Nelson [EMAIL PROTECTED]> 
added the comment:

Hey, if we can get rid of it, that's great.  You sound 95% 
certain in
your last post that completely removing test_socket_ssl on trunk and
py3k is the Right Thing -- any chance of getting you to 100% so I can
go ahead and delete it and close this ticket?

__
Tracker [EMAIL PROTECTED]>
http://bugs.python.org/issue1489>
__

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

[issue1489] test_socket_ssl hanhs on Windows (deadlock)

[Trent Nelson]

Trent Nelson <[EMAIL PROTECTED]> added the comment:

Hey, if we can get rid of it, that's great.  You sound 95% certain in 
your last post that completely removing test_socket_ssl on trunk and 
py3k is the Right Thing -- any chance of getting you to 100% so I can 
go ahead and delete it and close this ticket?

__
Tracker <[EMAIL PROTECTED]>

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

[issue1489] test_socket_ssl hanhs on Windows (deadlock)

[Bill Janssen]

Bill Janssen <[EMAIL PROTECTED]> added the comment:

Fine with me.  I don't have a checkout of the code handy, but I believe
this is obsolete test code.  All the SSL tests are, or should be, in
test_ssl, for 2.6 and 3.x.  The right fix is to remove it.

__
Tracker <[EMAIL PROTECTED]>

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

[issue1489] test_socket_ssl hanhs on Windows (deadlock)

[Trent Nelson]

Trent Nelson <[EMAIL PROTECTED]> added the comment:

Re-assigned this to myself (hope you don't mind janssen ;-) as I'm 
actively looking into it.  We build openssl.exe as part of our Windows 
build so we should make an effort to use this where possible.  Note 
that there is definitely a problem with 'openssl s_server' on Windows, 
I'm not sure if it's to do with buffering though as I can step through 
the same actions the test is performing in the console and I 
get 'ERROR\n' immediately back from the server.  This sort of falls 
into my existing work with issue 2550 (and patches) to improve network-
oriented tests such that they can be ran in parallel.  Note that patch 
trunk.2550-2.patch attached to issue2550 goes as far as converting 
openssl to use the -accept [port] option such that we don't bind on 
4443 by default (thus impeding other test_ssl_socket tests that may be 
running).

Also, openssl.exe in the s_server mode is using SO_REUSEADDR when it 
should be using SO_EXCLUSIVEADDRUSE.  That's not our problem per se, 
but given that we maintain our own openssl-0.9.8g sources, I'm 
considering fixing it if it improves the reliability of the test.  
(I'll feed back any changes to the OpenSSL project, of course.)

--
assignee: janssen -> Trent.Nelson
nosy: +Trent.Nelson

__
Tracker <[EMAIL PROTECTED]>

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

[issue1489] test_socket_ssl hanhs on Windows (deadlock)

[vila]

Changes by vila:


--
nosy: +vila

__
Tracker <[EMAIL PROTECTED]>

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

[issue1489] test_socket_ssl hanhs on Windows (deadlock)

[Guido van Rossum]

Changes by Guido van Rossum:


--
assignee:  -> janssen
nosy: +janssen

__
Tracker <[EMAIL PROTECTED]>

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

[issue1489] test_socket_ssl hanhs on Windows (deadlock)

[Paul Moore]

New submission from Paul Moore:

When running the test suite on Windows, test_socket_ssl hangs.

After a bit of investigation, it appears that the test is hanging at
line 184 (if self.s.stdout.readline() != "ERROR\n":) in
OpenSSLServer._external.

The problem is that the test assumes it can read a line of stdout from
the openssl.exe process. However, the openssl.exe on my PC (from the
GnuWin32 project) appears to buffer its output, so the Python process
deadlocks waiting for a response.

An easy (if clumsy) fix is to simply skip the check on the output of the
server. I have attached a patch which does this for win32 only (on the
assumption that other platforms don't have this issue).

I guess that other openssl builds don't have the same problem - however,
I can't see an easy way of testing for this.

--
components: Tests
files: test_socket_ssl.diff
messages: 57768
nosy: pmoore
severity: normal
status: open
title: test_socket_ssl hanhs on Windows (deadlock)
versions: Python 2.6
Added file: http://bugs.python.org/file8796/test_socket_ssl.diff

__
Tracker <[EMAIL PROTECTED]>

__

test_socket_ssl.diff
Description: Binary data
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com