Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

log sso name and email upon 500 error #53

Merged
merged 1 commit into from

3 participants

@benilovj
Owner

This is so that in the case of failure, it is possible to identify and follow up
with the person whose ticket creation failed.

@benilovj benilovj log sso name and email upon 500 error
This is so that in the case of failure, it is possible to identify and follow up
with the person whose ticket creation failed.
ca1fb13
@benilovj
Owner

@fatbusinessman do you see any problems with this?

@jamiecobbett jamiecobbett merged commit a41884b into from
@jamiecobbett jamiecobbett deleted the branch
@fatbusinessman

Nope, looks sensible to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 25, 2013
  1. @benilovj

    log sso name and email upon 500 error

    benilovj authored
    This is so that in the case of failure, it is possible to identify and follow up
    with the person whose ticket creation failed.
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 0 deletions.
  1. +1 −0  app/controllers/requests_controller.rb
View
1  app/controllers/requests_controller.rb
@@ -38,6 +38,7 @@ def raise_ticket(ticket)
return render "support/zendesk_error", locals: { error_string: "Zendesk timed out", ticket: ticket }
end
rescue GDSZendesk::ZendeskError => e
+ request.env["sso-credentials"] = "#{current_user.name} <#{current_user.email}>"
ExceptionNotifier::Notifier.exception_notification(request.env, e).deliver
render "support/zendesk_error", status: 500, locals: { error_string: e.underlying_message,
ticket: ticket }
Something went wrong with that request. Please try again.