Improve the TLS intolerant server testing support added in r134129

net/tools/testserver/testserver.py

Index: net/tools/testserver/testserver.py
===================================================================
--- net/tools/testserver/testserver.py	(revision 138367)
+++ net/tools/testserver/testserver.py	(working copy)
@@ -2177,9 +2177,12 @@
                            'automatically generated certificate. One of '
                            '[ok,revoked,invalid]')
   option_parser.add_option('', '--tls-intolerant', dest='tls_intolerant',

[wtc, 2012/05/22 22:02:02]

I am not familiar with Python.  I found that if I remove
    dest='tls_intolerant',
here, things still seem to work.  Do you know why?

[agl, 2012/05/22 22:07:11]

According to:

http://docs.python.org/library/optparse.html#the-store-action

The default destination is built by replacing hyphens with underscores in the
long-form argument. So dest can be safely omitted in this option.

-                           const=True, default=False, action='store_const',
-                           help='If true, TLS connections will be aborted '
-                           ' in order to test SSLv3 fallback.')
+                           default='0', type='int',
+                           help='If nonzero, TLS connections will be aborted '
+                           ' in order to test version fallback. 1 means all'
+                           ' TLS versions will be aborted. 2 means TLS 1.1'
+                           ' or higher will be aborted. 3 means TLS 1.2 or'
+                           ' higher will be aborted.')
   option_parser.add_option('', '--https-record-resume', dest='record_resume',
                            const=True, default=False, action='store_const',
                            help='Record resumption cache events rather than'