Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(128)

Unified Diff: tools/md_browser/md_browser.py

Issue 2875633003: Default md_browser.py to listen just on localhost (Closed)
Patch Set: remove flag Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/md_browser/md_browser.py
diff --git a/tools/md_browser/md_browser.py b/tools/md_browser/md_browser.py
index 8ed405732aed3b0d55ed7330d3e5856fed49410a..ee9d9e2aa5e8cd6d38b6ba1bce5dd00b1b2f23dc 100755
--- a/tools/md_browser/md_browser.py
+++ b/tools/md_browser/md_browser.py
@@ -36,10 +36,10 @@ def main(argv):
args = parser.parse_args(argv)
top_level = os.path.realpath(args.directory)
+ server_address = ('localhost', args.port)
+ s = Server(server_address, top_level)
- s = Server(args.port, top_level)
-
- print('Listening on http://localhost:%s/' % args.port)
+ print('Listening on http://%s:%s/' % server_address)
thread = None
if args.file:
path = os.path.realpath(args.file)
@@ -106,11 +106,9 @@ def _gitiles_slugify(value, _separator):
class Server(SocketServer.TCPServer):
- def __init__(self, port, top_level):
- SocketServer.TCPServer.__init__(self, ('0.0.0.0', port), Handler)
- self.port = port
+ def __init__(self, server_address, top_level):
+ SocketServer.TCPServer.__init__(self, server_address, Handler)
self.top_level = top_level
- self.retcode = None
def server_bind(self):
self.socket.setsockopt(socket.SOL_SOCKET, socket.SO_REUSEADDR, 1)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698