openstack/deb-python-falcon
Code Issues Proposed changes

docs(tutorial): Use mimetypes from standard lib

Browse Source
This commit is contained in:
David Larlet
2016-03-14 15:14:33 +01:00
parent ac03888ad7
commit 1963af7438
1 changed files with 8 additions and 36 deletions
Download Patch File Download Diff File Expand all files Collapse all files

44
doc/user/tutorial.rst
View File

@@ -253,31 +253,20 @@ Next, let's implement the POST responder:
.. code:: python
import os
import time
import uuid
import mimetypes
import falcon
def _media_type_to_ext(media_type):
# Strip off the 'image/' prefix
return media_type[6:]
def _generate_id():
return str(uuid.uuid4())
class Resource(object):
def __init__(self, storage_path):
self.storage_path = storage_path
def on_post(self, req, resp):
image_id = _generate_id()
ext = _media_type_to_ext(req.content_type)
filename = image_id + '.' + ext
ext = mimetypes.guess_extension(req.content_type)
filename = '{uuid}{ext}'.format(uuid=uuid.uuid4(), ext=ext)
image_path = os.path.join(self.storage_path, filename)
with open(image_path, 'wb') as image_file:
@@ -289,7 +278,7 @@ Next, let's implement the POST responder:
image_file.write(chunk)
resp.status = falcon.HTTP_201
resp.location = '/images/' + image_id
resp.location = '/images/' + filename
As you can see, we generate a unique ID and filename for the new image, and
then write it out by reading from ``req.stream``. It's called ``stream`` instead
@@ -350,35 +339,20 @@ Go ahead and edit your ``images.py`` file to look something like this:
.. code:: python
import os
import time
import uuid
import mimetypes
import falcon
def _media_type_to_ext(media_type):
# Strip off the 'image/' prefix
return media_type[6:]
def _ext_to_media_type(ext):
return 'image/' + ext
def _generate_id():
return str(uuid.uuid4())
class Collection(object):
def __init__(self, storage_path):
self.storage_path = storage_path
def on_post(self, req, resp):
image_id = _generate_id()
ext = _media_type_to_ext(req.content_type)
filename = image_id + '.' + ext
ext = mimetypes.guess_extension(req.content_type)
filename = '{uuid}{ext}'.format(uuid=uuid.uuid4(), ext=ext)
image_path = os.path.join(self.storage_path, filename)
with open(image_path, 'wb') as image_file:
@@ -399,9 +373,7 @@ Go ahead and edit your ``images.py`` file to look something like this:
self.storage_path = storage_path
def on_get(self, req, resp, name):
ext = os.path.splitext(name)[1][1:]
resp.content_type = _ext_to_media_type(ext)
resp.content_type = mimetypes.guess_type(name)[0]
image_path = os.path.join(self.storage_path, name)
resp.stream = open(image_path, 'rb')
resp.stream_len = os.path.getsize(image_path)