vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: email: get unsub email: use cstr instead of encode (#20985)

Browse source 
`encode` fails if reference name is an integer. use `cstr` instead which
use `as_unicode` under the hood
This commit is contained in:
Sabu Siyad 2023-05-14 17:29:38 +05:30 • committed by GitHub
parent 4bd32bcf04
commit 2ec893278a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
frappe/email/queue.py
View file

@ -3,7 +3,7 @@
import frappe
from frappe import _, msgprint
from frappe.utils import cint, get_url, now_datetime
from frappe.utils import cint, cstr, get_url, now_datetime
from frappe.utils.data import getdate
from frappe.utils.verified_command import get_signed_params, verify_request
@ -81,9 +81,9 @@ def get_unsubcribed_url(
reference_doctype, reference_name, email, unsubscribe_method, unsubscribe_params
):
params = {
"email": email.encode("utf-8"),
"doctype": reference_doctype.encode("utf-8"),
"name": reference_name.encode("utf-8"),
"email": cstr(email),
"doctype": cstr(reference_doctype),
"name": cstr(reference_name),
}
if unsubscribe_params:
params.update(unsubscribe_params)