Commit bb0effb7 by Clinton Blackburn

Updated Zendesk error handling

Properly logging exceptions so we have the complete stacktrace.

ECOM-2792
parent e46b4396
...@@ -175,8 +175,8 @@ def _record_feedback_in_zendesk(realname, email, subject, details, tags, additio ...@@ -175,8 +175,8 @@ def _record_feedback_in_zendesk(realname, email, subject, details, tags, additio
} }
try: try:
ticket_id = zendesk_api.create_ticket(new_ticket) ticket_id = zendesk_api.create_ticket(new_ticket)
except zendesk.ZendeskError as err: except zendesk.ZendeskError:
log.error("Error creating Zendesk ticket: %s", str(err)) log.exception("Error creating Zendesk ticket")
return False return False
# Additional information is provided as a private update so the information # Additional information is provided as a private update so the information
...@@ -184,8 +184,8 @@ def _record_feedback_in_zendesk(realname, email, subject, details, tags, additio ...@@ -184,8 +184,8 @@ def _record_feedback_in_zendesk(realname, email, subject, details, tags, additio
ticket_update = {"ticket": {"comment": {"public": False, "body": additional_info_string}}} ticket_update = {"ticket": {"comment": {"public": False, "body": additional_info_string}}}
try: try:
zendesk_api.update_ticket(ticket_id, ticket_update) zendesk_api.update_ticket(ticket_id, ticket_update)
except zendesk.ZendeskError as err: except zendesk.ZendeskError:
log.error("Error updating Zendesk ticket: %s", str(err)) log.exception("Error updating Zendesk ticket")
# The update is not strictly necessary, so do not indicate failure to the user # The update is not strictly necessary, so do not indicate failure to the user
pass pass
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment