Mercurial > hg > uploader
annotate uploader/handlers.py @ 15:1ee374416987
handle more gracefully when no file is uploaded
| author | Jeff Hammel <jhammel@mozilla.com> |
|---|---|
| date | Tue, 12 Jul 2011 09:18:29 -0700 |
| parents | 916d45d4f921 |
| children | d15f85eb2ab9 |
| rev | line source |
|---|---|
| 0 | 1 import os |
| 2 from urlparse import urlparse | |
| 3 from webob import Response, exc | |
| 4 | |
| 5 class Handler(object): | |
| 6 def __init__(self, app, request): | |
| 7 self.app = app | |
| 8 self.request = request | |
| 9 self.application_path = urlparse(request.application_url)[2] | |
| 10 | |
| 11 def link(self, path=(), permanant=False): | |
| 12 if isinstance(path, basestring): | |
| 13 path = [ path ] | |
| 14 path = [ i.strip('/') for i in path ] | |
| 15 if permanant: | |
| 16 application_url = [ self.request.application_url ] | |
| 17 else: | |
| 18 application_url = [ self.application_path ] | |
| 19 path = application_url + path | |
| 20 return '/'.join(path) | |
| 21 | |
| 22 def redirect(self, location): | |
|
13
b8c636b0b567
make work for non pastescript frameworks
Jeff Hammel <jhammel@mozilla.com>
parents:
11
diff
changeset
|
23 return exc.HTTPSeeOther(location=location) |
| 0 | 24 |
| 25 class Get(Handler): | |
| 26 | |
|
13
b8c636b0b567
make work for non pastescript frameworks
Jeff Hammel <jhammel@mozilla.com>
parents:
11
diff
changeset
|
27 form = """<form name="upload_form" method="post" enctype="multipart/form-data"> |
| 2 | 28 <input type="file" name="file"/><input type="submit" value="upload"/></form></body></html>""" |
| 29 | |
| 0 | 30 @classmethod |
| 2 | 31 def match(cls, app, request): |
|
7
a8c60480fce0
make query_string argument more universal
k0s <k0scist@gmail.com>
parents:
6
diff
changeset
|
32 if app.query_string and (app.query_string not in request.GET): |
|
a8c60480fce0
make query_string argument more universal
k0s <k0scist@gmail.com>
parents:
6
diff
changeset
|
33 return False |
| 0 | 34 return request.method == 'GET' |
| 35 | |
| 36 def __call__(self): | |
|
13
b8c636b0b567
make work for non pastescript frameworks
Jeff Hammel <jhammel@mozilla.com>
parents:
11
diff
changeset
|
37 form = "<html><body>" |
|
b8c636b0b567
make work for non pastescript frameworks
Jeff Hammel <jhammel@mozilla.com>
parents:
11
diff
changeset
|
38 if 'uploaded' in self.request.GET: |
|
b8c636b0b567
make work for non pastescript frameworks
Jeff Hammel <jhammel@mozilla.com>
parents:
11
diff
changeset
|
39 form += '<div>%s uploaded successfully</div>' % self.request.GET['uploaded'] |
|
b8c636b0b567
make work for non pastescript frameworks
Jeff Hammel <jhammel@mozilla.com>
parents:
11
diff
changeset
|
40 form += self.form + '</body></html>' |
|
b8c636b0b567
make work for non pastescript frameworks
Jeff Hammel <jhammel@mozilla.com>
parents:
11
diff
changeset
|
41 return Response(content_type='text/html', body=form) |
| 0 | 42 |
| 43 class Post(Handler): | |
| 44 | |
| 45 @classmethod | |
| 2 | 46 def match(cls, app, request): |
| 0 | 47 return request.method == 'POST' |
| 48 | |
| 2 | 49 def write(self, fin, path): |
| 50 fout = file(path, 'w') | |
| 51 fout.write(fin.file.read()) | |
| 52 fout.close() | |
| 53 | |
| 0 | 54 def __call__(self): |
| 55 fin = self.request.POST['file'] | |
|
15
1ee374416987
handle more gracefully when no file is uploaded
Jeff Hammel <jhammel@mozilla.com>
parents:
14
diff
changeset
|
56 try: |
|
1ee374416987
handle more gracefully when no file is uploaded
Jeff Hammel <jhammel@mozilla.com>
parents:
14
diff
changeset
|
57 _path = fin.filename.replace('..', '_') |
|
1ee374416987
handle more gracefully when no file is uploaded
Jeff Hammel <jhammel@mozilla.com>
parents:
14
diff
changeset
|
58 except AttributeError: # no file uploaded |
|
1ee374416987
handle more gracefully when no file is uploaded
Jeff Hammel <jhammel@mozilla.com>
parents:
14
diff
changeset
|
59 return self.redirect(self.link('/')) |
|
14
916d45d4f921
dont just die on questionable filenames
Jeff Hammel <jhammel@mozilla.com>
parents:
13
diff
changeset
|
60 _path = _path.replace(os.path.sep, '_') |
|
916d45d4f921
dont just die on questionable filenames
Jeff Hammel <jhammel@mozilla.com>
parents:
13
diff
changeset
|
61 _path = os.path.join(self.app.directory, _path) |
| 11 | 62 self.write(fin, _path) |
|
13
b8c636b0b567
make work for non pastescript frameworks
Jeff Hammel <jhammel@mozilla.com>
parents:
11
diff
changeset
|
63 return self.redirect(self.link('/?uploaded=' + fin.filename)) |
| 2 | 64 |
| 65 def path(directory, request): | |
| 66 if os.sep == '/': | |
| 67 return os.path.join(directory, request.path_info.strip('/')) | |
| 68 return os.path.join(directory, *request.path_info.strip('/').split('/')) | |
| 69 | |
| 70 class SubpathGet(Get): | |
| 71 | |
| 72 @classmethod | |
| 73 def match(cls, app, request): | |
| 9 | 74 if not Get.match(app, request): |
| 2 | 75 return False |
| 76 _path = path(app.directory, request) | |
| 77 if os.path.exists(_path) and os.path.isdir(_path): | |
| 78 return True | |
| 0 | 79 |
| 2 | 80 class SubpathPost(Post): |
| 81 | |
| 82 @classmethod | |
| 83 def match(cls, app, request): | |
| 84 if request.method != 'POST': | |
| 85 return False | |
| 86 _path = path(app.directory, request) | |
| 87 if os.path.exists(_path) and os.path.isdir(_path): | |
| 88 return True | |
| 89 | |
| 90 def __call__(self): | |
| 91 fin = self.request.POST['file'] | |
| 10 | 92 _path = path(self.app.directory, self.request) |
| 93 _path = os.path.join(_path, fin.filename) | |
| 94 self.write(fin, _path) | |
|
13
b8c636b0b567
make work for non pastescript frameworks
Jeff Hammel <jhammel@mozilla.com>
parents:
11
diff
changeset
|
95 return self.redirect(self.link(self.request.path_info)) |
| 2 | 96 |