Remove completed gettor requests from database

This will remove gettor requests from the requests database when they
are completed (rather than changing the status to 'SENT'). There's no
need to keep this information around, and we don't want to store
hashed email addresses as identifiers long term.

gettor/services/email/sendmail.py

@@ -191,9 +191,8 @@ class Sendmail(object):
                         service="email"
                     )
 
-                    yield self.conn.update_request(
-                        id=id, hid=hid.hexdigest(), status="SENT",
-                        service="email", date=date
+                    yield self.conn.remove_request(
+                        id=id, service="email", date=date
                     )
 
             except SMTPError as e:
@@ -245,9 +244,8 @@ class Sendmail(object):
                         service="email"
                     )
 
-                    yield self.conn.update_request(
-                        id=id, hid=hid.hexdigest(), status="SENT",
-                        service="email", date=date
+                    yield self.conn.remove_request(
+                        id=id, service="email", date=date
                     )
 
             except SMTPError as e:

gettor/utils/db.py

@@ -72,15 +72,14 @@ class SQLite3(object):
 			query, (id, service)
 		).addCallback(self.query_callback).addErrback(self.query_errback)
 
-	def update_request(self, id, hid, status, service, date):
+	def remove_request(self, id, service, date):
 		"""
-		Update request record in the database
+		Removes completed request record from the database
 		"""
-		query = "UPDATE requests SET id=?, status=? WHERE id=? AND "\
-		"service=? AND date=?"
+		query = "DELETE FROM requests WHERE id=? AND service=? AND date=?"
 
 		return self.dbpool.runQuery(
-			query, (hid, status, id, service, date)
+			query, (id, service, date)
 		).addCallback(self.query_callback).addErrback(self.query_errback)
 
 	def update_stats(self, command, service, platform=None, language='en'):