nik/FreedomBox
1
0
Fork 0
mirror of https://github.com/freedombox/FreedomBox.git synced 2026-04-15 09:51:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

email_server: aliases: Drop unnecessary sanitizing

Browse Source 
- In the list form, the values are already sanitized as they are retrieved from
the database. Don't refuse to manage existing aliases that don't fit the format.

- In the create form, the form already sanitizes as necessary.

Signed-off-by: Sunil Mohan Adapa <sunil@medhas.org>
Reviewed-by: James Valleroy <jvalleroy@mailbox.org>
This commit is contained in:
Sunil Mohan Adapa 2021-10-13 16:55:36 -07:00 committed by James Valleroy
parent 6e236a41a8
commit c89a7fdb9d
No known key found for this signature in database
GPG Key ID: 77C0C75E7B650808
1 changed files with 0 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
plinth/modules/email_server/aliases/__init__.py
View File

@ -71,7 +71,6 @@ def put(uid_number, email_name):
SELECT ?,?,? WHERE NOT EXISTS(
SELECT 1 FROM Alias WHERE email_name=?
)"""
email_name = models.sanitize_email_name(email_name)
with db_cursor() as cur:
cur.execute(s, (email_name, uid_number, 1, email_name))
@ -80,9 +79,6 @@ def put(uid_number, email_name):
def delete(uid_number, alias_list):
s = 'DELETE FROM Alias WHERE uid_number=? AND email_name=?'
for i in range(len(alias_list)):
alias_list[i] = models.sanitize_email_name(alias_list[i])
parameter_seq = ((uid_number, a) for a in alias_list)
with db_cursor() as cur:
cur.execute('BEGIN')
@ -101,9 +97,6 @@ def set_disabled(uid_number, alias_list):
def _set_status(uid_number, alias_list, status):
s = 'UPDATE Alias SET status=? WHERE uid_number=? AND email_name=?'
for i in range(len(alias_list)):
alias_list[i] = models.sanitize_email_name(alias_list[i])
parameter_seq = ((status, uid_number, a) for a in alias_list)
with db_cursor() as cur:
cur.execute('BEGIN')