summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStéphane Wirtel <stephane@wirtel.be>2019-04-16 12:52:54 (GMT)
committerGitHub <noreply@github.com>2019-04-16 12:52:54 (GMT)
commit2b7f93b99afbe78e4e567d9252d9470d29f387c8 (patch)
treed55ea2d06b9c3f31d3011ae66a583f011e8c8963
parent6fa84bd12c4b83bee6a41b989363230d5c03b96c (diff)
downloadcpython-2b7f93b99afbe78e4e567d9252d9470d29f387c8.zip
cpython-2b7f93b99afbe78e4e567d9252d9470d29f387c8.tar.gz
cpython-2b7f93b99afbe78e4e567d9252d9470d29f387c8.tar.bz2
bpo-36345: Update wsgiref example (GH-12562)
Use literalinclude markup to include Tools/scripts/serve.py code. Tools/scripts/serve.py first argument on the command line is now optional.
-rw-r--r--Doc/library/wsgiref.rst34
-rw-r--r--Misc/NEWS.d/next/Documentation/2019-03-26-14-58-34.bpo-36345.r2stx3.rst3
-rwxr-xr-xTools/scripts/serve.py5
3 files changed, 10 insertions, 32 deletions
diff --git a/Doc/library/wsgiref.rst b/Doc/library/wsgiref.rst
index b85ec53..ec51367 100644
--- a/Doc/library/wsgiref.rst
+++ b/Doc/library/wsgiref.rst
@@ -783,33 +783,7 @@ This is a working "Hello World" WSGI application::
httpd.serve_forever()
-Example of a small wsgiref-based web server::
-
- # Takes a path to serve from and an optional port number (defaults to 8000),
- # then tries to serve files. Mime types are guessed from the file names, 404
- # errors are raised if the file is not found.
- import sys
- import os
- import mimetypes
- from wsgiref import simple_server, util
-
- def app(environ, respond):
- fn = os.path.join(path, environ['PATH_INFO'][1:])
- if '.' not in fn.split(os.path.sep)[-1]:
- fn = os.path.join(fn, 'index.html')
- type = mimetypes.guess_type(fn)[0]
-
- if os.path.exists(fn):
- respond('200 OK', [('Content-Type', type)])
- return util.FileWrapper(open(fn, "rb"))
- else:
- respond('404 Not Found', [('Content-Type', 'text/plain')])
- return [b'not found']
-
- path = sys.argv[1]
- port = int(sys.argv[2]) if len(sys.argv) > 2 else 8000
- with simple_server.make_server('', port, app) as httpd:
- print("Serving {} on port {}, control-C to stop".format(path, port))
-
- # Serve until process is killed
- httpd.serve_forever()
+Example of a WSGI application serving the current directory, accept optional
+directory and port number (default: 8000) on the command line:
+
+.. literalinclude:: ../../Tools/scripts/serve.py
diff --git a/Misc/NEWS.d/next/Documentation/2019-03-26-14-58-34.bpo-36345.r2stx3.rst b/Misc/NEWS.d/next/Documentation/2019-03-26-14-58-34.bpo-36345.r2stx3.rst
new file mode 100644
index 0000000..bbecc94
--- /dev/null
+++ b/Misc/NEWS.d/next/Documentation/2019-03-26-14-58-34.bpo-36345.r2stx3.rst
@@ -0,0 +1,3 @@
+Avoid the duplication of code from ``Tools/scripts/serve.py`` in using the
+:rst:dir:`literalinclude` directive for the basic wsgiref-based web server in the
+documentation of :mod:`wsgiref`. Contributed by Stéphane Wirtel.
diff --git a/Tools/scripts/serve.py b/Tools/scripts/serve.py
index dae21f2..7ac9c10 100755
--- a/Tools/scripts/serve.py
+++ b/Tools/scripts/serve.py
@@ -25,11 +25,12 @@ def app(environ, respond):
return [b'not found']
if __name__ == '__main__':
- path = sys.argv[1]
+ path = sys.argv[1] if len(sys.argv) > 1 else os.getcwd()
port = int(sys.argv[2]) if len(sys.argv) > 2 else 8000
httpd = simple_server.make_server('', port, app)
print("Serving {} on port {}, control-C to stop".format(path, port))
try:
httpd.serve_forever()
except KeyboardInterrupt:
- print("\b\bShutting down.")
+ print("Shutting down.")
+ httpd.server_close()