Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: sitescripts/crashes/web/submitCrash.py

Issue 8358150: Crash submission backend (Closed)
Patch Set: Created Sept. 17, 2012, 2:47 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sitescripts/crashes/web/__init__.py ('k') | sitescripts/web.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/crashes/web/submitCrash.py
===================================================================
new file mode 100644
--- /dev/null
+++ b/sitescripts/crashes/web/submitCrash.py
@@ -0,0 +1,55 @@
+# coding: utf-8
+
+# This Source Code is subject to the terms of the Mozilla Public License
+# version 2.0 (the "License"). You can obtain a copy of the License at
+# http://mozilla.org/MPL/2.0/.
+
+import re, os, sys
+from urlparse import parse_qs
+from tempfile import mkstemp
+from sitescripts.utils import get_config, setupStderr
+from sitescripts.web import url_handler
+
+@url_handler('/submitCrash')
+def handleRequest(environ, start_response):
+ setupStderr(environ['wsgi.errors'])
+
+ if not environ.get('HTTP_X_ADBLOCK_PLUS'):
+ return showError('Please use Adblock Plus to submit crashes', start_response)
+
+ if environ['REQUEST_METHOD'].upper() != 'POST' or not environ.get('CONTENT_TYPE', '').startswith('text/xml'):
+ return showError('Unsupported request method', start_response)
+
+ params = parse_qs(environ.get('QUERY_STRING', ''))
+
+ requestVersion = params.get('version', ['0'])[0]
+ if requestVersion != '1':
+ return showError('Unsupported request version', start_response)
+
+ try:
+ request_body_size = int(environ.get('CONTENT_LENGTH', 0))
+ except (ValueError):
+ return showError('No content', start_response)
+
+ dir = get_config().get('crashes', 'dataPath')
+ if not os.path.exists(dir):
+ os.makedirs(dir)
+
+ filename = None
+ try:
+ fd, filename = mkstemp('.xml.tmp', 'crash_', dir)
+ file = os.fdopen(fd, 'w')
Wladimir Palant 2012/09/17 15:15:16 'wb' please, we don't want the data to be changed.
Andrey Novikov 2012/09/17 15:33:24 Done.
+ file.write(environ['wsgi.input'].read(request_body_size))
+ file.close()
+ os.rename(filename, filename[:-4]);
Wladimir Palant 2012/09/17 15:15:16 Please use os.path.splitext(), this will make the
Andrey Novikov 2012/09/17 15:33:24 Done.
+ except Exception, e:
+ if filename != None and os.path.isfile(filename):
+ os.remove(filename)
+ raise e
+
+ start_response('200 Ok', [('Content-Type', 'text/plain; charset=utf-8')])
+ return ['saved'.encode('utf-8')]
+
+def showError(message, start_response):
+ start_response('400 Processing Error', [('Content-Type', 'text/plain; charset=utf-8')])
+ return [message.encode('utf-8')]
« no previous file with comments | « sitescripts/crashes/web/__init__.py ('k') | sitescripts/web.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld