all details in email too, still just for CodeError

svn path=/trunk/packages/sipb-xen-www/; revision=601

diff --git a/code/main.py b/code/main.py
index 01d6e39..dd6b867 100755 (executable)
--- a/code/main.py
+++ b/code/main.py
@@ -119,11 +119,12 @@ DEFAULT_HEADERS = {'Content-Type': 'text/html'}
 
 def error(op, username, fields, err, emsg, traceback):
     """Print an error page when a CodeError occurs"""
 
 def error(op, username, fields, err, emsg, traceback):
     """Print an error page when a CodeError occurs"""

-    send_error_mail('xvm error on %s for %s: %s' % (op, username, err),
-                    'error on %s for %s: %s\n\n%s\n'
-                    % (op, username, err, emsg))

     d = dict(op=op, user=username, fields=fields,
              errorMessage=str(err), stderr=emsg, traceback=traceback)
     d = dict(op=op, user=username, fields=fields,
              errorMessage=str(err), stderr=emsg, traceback=traceback)

+    details = templates.error_raw(searchList=[d])
+    send_error_mail('xvm error on %s for %s: %s' % (op, username, err),
+                    details)
+    d['details'] = details

     return templates.error(searchList=[d])
 
 def invalidInput(op, username, fields, err, emsg):
     return templates.error(searchList=[d])
 
 def invalidInput(op, username, fields, err, emsg):

diff --git a/code/templates/__init__.py b/code/templates/__init__.py
index d26672f..b2b23da 100644 (file)
--- a/code/templates/__init__.py
+++ b/code/templates/__init__.py
@@ -1,4 +1,4 @@
-__all__ = 'info command error help invalid list unauth vnc'.split()
+__all__ = 'info command error error_raw help invalid list unauth vnc'.split()

 for _name in __all__:
     try:
         _module = __import__(_name, globals(), {}, [_name])
 for _name in __all__:
     try:
         _module = __import__(_name, globals(), {}, [_name])

diff --git a/code/templates/error.tmpl b/code/templates/error.tmpl
index b11deaa..ff9b4e0 100644 (file)
--- a/code/templates/error.tmpl
+++ b/code/templates/error.tmpl
@@ -12,21 +12,9 @@ mail about it.</p>
 <p>Feel free to poke us at <tt>xvm@mit.edu</tt> if this bug is
 consistently biting you and we don't seem to be fixing it.</p>
 
 <p>Feel free to poke us at <tt>xvm@mit.edu</tt> if this bug is
 consistently biting you and we don't seem to be fixing it.</p>
 

-<p>In case you're curious, the gory details are here.</p>
+<p>In case you're curious, the gory details are below.</p>

 
 <pre>
 
 <pre>

-Error on operation $op for user $user: $errorMessage
-
-Fields:
-#for $f in $fields:
-$f=$fields[$f].value
-#end for
-
-Error output:
-$stderr#slurp
----- end error output
-
-Traceback:
-$traceback
+$details

 </pre>
 #end def
 </pre>
 #end def

diff --git a/code/templates/error_raw.tmpl b/code/templates/error_raw.tmpl
new file mode 100644 (file)
index 0000000..bd4db84
--- /dev/null
+++ b/code/templates/error_raw.tmpl
@@ -0,0 +1,13 @@
+Error on operation $op for user $user: $errorMessage
+
+Fields:
+#for $f in $fields:
+$f=$fields[$f].value
+#end for
+
+Error output:
+$stderr#slurp
+---- end error output
+
+Traceback:
+$traceback