fix semgrep issues

Author: trufurs

frappe_gmail_thread/api/activity.py

@@ -16,7 +16,7 @@ def get_attachments_data(email):
 
 
 @frappe.whitelist()
-def get_linked_gmail_threads(doctype, docname):
+def get_linked_gmail_threads(doctype: str, docname: str):
     gmail_threads = frappe.get_all(
         "Gmail Thread",
         filters={
@@ -83,7 +83,7 @@ def get_linked_gmail_threads(doctype, docname):
 
 
 @frappe.whitelist()
-def relink_gmail_thread(name, doctype, docname):
+def relink_gmail_thread(name: str, doctype: str, docname: str):
     thread = frappe.get_doc("Gmail Thread", name)
     thread.reference_doctype = doctype
     thread.reference_name = docname
@@ -92,7 +92,7 @@ def relink_gmail_thread(name, doctype, docname):
 
 
 @frappe.whitelist()
-def unlink_gmail_thread(name):
+def unlink_gmail_thread(name: str):
     thread = frappe.get_doc("Gmail Thread", name)
     thread.reference_doctype = None
     thread.reference_name = None

frappe_gmail_thread/api/oauth.py

@@ -20,7 +20,7 @@ def get_authentication_url(client_id=None, redirect_uri=None):
 
 
 @frappe.whitelist()
-def get_auth_url(doc):
+def get_auth_url(doc: str):
     current_user = frappe.session.user
     current_user = frappe.get_doc("User", current_user)
     try:
@@ -115,7 +115,7 @@ def authorize_access(user, code=None, reauthorize=None):
 
 
 @frappe.whitelist()
-def callback(code):
+def callback(code: str):
     """
     Authorization code is sent to callback as per the API configuration
     """

frappe_gmail_thread/api/pubsub.py

@@ -5,7 +5,7 @@
 from frappe.utils.background_jobs import is_job_enqueued
 
 
-@frappe.whitelist(allow_guest=True)
+@frappe.whitelist(allow_guest=True)  # nosemgrep
 def callback():
     data = frappe.request.get_data(as_text=True)
     data = frappe.parse_json(data)

frappe_gmail_thread/frappe_gmail_thread/doctype/gmail_thread/gmail_thread.py

@@ -91,7 +91,7 @@ def before_save(self):
 
 
 @frappe.whitelist(methods=["POST"])
-def sync_labels(account_name, should_save=True):
+def sync_labels(account_name: str, should_save: bool = True):
     if isinstance(account_name, str):
         gmail_account = frappe.get_doc("Gmail Account", account_name)
     else:
@@ -116,7 +116,7 @@ def sync_labels(account_name, should_save=True):
 
 def sync(user=None):
     if user:
-        frappe.set_user(user)
+        frappe.set_user(user)  # nosemgrep:
     user = frappe.session.user
     gmail_account = frappe.get_doc("Gmail Account", {"linked_user": user})
     if not gmail_account.gmail_enabled: