mirrors/debugpy
1
0
Fork 0
mirror of https://github.com/microsoft/debugpy.git synced 2025-12-23 08:48:12 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Always use a localhost addr in debug().

Browse source
This commit is contained in:
Eric Snow 2018-04-04 00:09:15 +00:00
parent 2f364e6171
commit 899d146dd7
2 changed files with 14 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

4
ptvsd/debugger.py
View file

@ -13,6 +13,8 @@ __version__ = "4.0.0a5"
# TODO: not needed?
DONT_DEBUG = []
LOCALHOST = '127.0.0.1'
RUNNERS = {
'module': run_module, # python -m spam
'script': run_file, # python spam.py
@ -26,7 +28,7 @@ def debug(filename, port_num, debug_id, debug_options, run_as,
# TODO: docstring
if _extra is None:
_extra = sys.argv[1:]
address = (None, port_num)
address = (LOCALHOST, port_num)
try:
run = _runners[run_as]
except KeyError:

17
tests/ptvsd/test_debugger.py
View file

@ -1,7 +1,7 @@
import sys
import unittest
from ptvsd.debugger import debug
from ptvsd.debugger import debug, LOCALHOST
class DebugTests(unittest.TestCase):
@ -26,7 +26,7 @@ class DebugTests(unittest.TestCase):
def test_module(self):
filename = 'spam'
_, port = addr = (None, 8888)
_, port = addr = (LOCALHOST, 8888)
debug_id = 1
debug_options = {'x': 'y'}
debug(filename, port, debug_id, debug_options, 'module',
@ -38,7 +38,7 @@ class DebugTests(unittest.TestCase):
def test_script(self):
filename = 'spam.py'
_, port = addr = (None, 8888)
_, port = addr = (LOCALHOST, 8888)
debug_id = 1
debug_options = {'x': 'y'}
debug(filename, port, debug_id, debug_options, 'script',
@ -50,7 +50,7 @@ class DebugTests(unittest.TestCase):
def test_code(self):
filename = "print('spam')"
_, port = addr = (None, 8888)
_, port = addr = (LOCALHOST, 8888)
debug_id = 1
debug_options = {'x': 'y'}
debug(filename, port, debug_id, debug_options, 'code',
@ -62,7 +62,7 @@ class DebugTests(unittest.TestCase):
def test_unsupported(self):
filename = 'spam'
_, port = addr = (None, 8888)
_, port = addr = (LOCALHOST, 8888)
debug_id = 1
debug_options = {'x': 'y'}
debug(filename, port, debug_id, debug_options, '???',
@ -74,7 +74,7 @@ class DebugTests(unittest.TestCase):
def test_extra_sys_argv(self):
filename = 'spam.py'
_, port = addr = (None, 8888)
_, port = addr = (LOCALHOST, 8888)
debug_id = 1
debug_options = {'x': 'y'}
extra = ['--eggs', 'abc']
@ -112,6 +112,7 @@ class IntegrationTests(unittest.TestCase):
self.assertEqual(self.argv, [
'eggs',
'--port', '8888',
'--client', LOCALHOST,
'--module',
'--file', 'spam:',
])
@ -129,6 +130,7 @@ class IntegrationTests(unittest.TestCase):
self.assertEqual(self.argv, [
'eggs',
'--port', '8888',
'--client', LOCALHOST,
'--file', 'spam.py',
])
self.assertEqual(self.kwargs, {})
@ -145,6 +147,7 @@ class IntegrationTests(unittest.TestCase):
self.assertEqual(self.argv, [
'eggs',
'--port', '8888',
'--client', LOCALHOST,
'--file', filename,
])
self.assertEqual(self.kwargs, {})
@ -161,6 +164,7 @@ class IntegrationTests(unittest.TestCase):
self.assertEqual(self.argv, [
'eggs',
'--port', '8888',
'--client', LOCALHOST,
'--file', 'spam',
])
self.assertEqual(self.kwargs, {})
@ -177,6 +181,7 @@ class IntegrationTests(unittest.TestCase):
self.assertEqual(self.argv, [
'eggs',
'--port', '8888',
'--client', LOCALHOST,
'--file', 'spam.py',
'--abc', 'xyz',
'42',