Prohibit +/-infinity as timeout values

This commit is contained in:
A. Jesse Jiryu Davis 2012-10-06 11:19:17 -07:00
parent 1da312ea43
commit 28a7ed9a93
2 changed files with 3 additions and 1 deletions

View File

@ -93,7 +93,7 @@ def validate_positive_float(option, value):
value = float(value)
except (ValueError, TypeError):
raise err
if value <= 0:
if not 0 < value < float('inf'):
raise err
return value

View File

@ -84,6 +84,8 @@ class TestURI(unittest.TestCase):
self.assertRaises(ConfigurationError, split_options, 'socketTimeoutMS=0.0')
self.assertRaises(ConfigurationError, split_options, 'connectTimeoutMS=foo')
self.assertRaises(ConfigurationError, split_options, 'connectTimeoutMS=0.0')
self.assertRaises(ConfigurationError, split_options, 'connectTimeoutMS=inf')
self.assertRaises(ConfigurationError, split_options, 'connectTimeoutMS=-inf')
self.assertTrue(split_options('socketTimeoutMS=300'))
self.assertTrue(split_options('connectTimeoutMS=300'))
self.assertEqual({'sockettimeoutms': 0.3}, split_options('socketTimeoutMS=300'))