diff options
author | Stan Hu <stanhu@gmail.com> | 2016-08-19 03:06:33 +0300 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2016-08-24 22:09:51 +0300 |
commit | 170885edd6f3ea52792511586778e0dce8021cf7 (patch) | |
tree | 4e7b04f41722dd4b369ecc7ed55285845e1e536f /lib/ci/api/api.rb | |
parent | 7b0b2417491e28e5536688e0fca96829a4cb7900 (diff) |
Add Sentry logging to API calls
Closes #21043
Diffstat (limited to 'lib/ci/api/api.rb')
-rw-r--r-- | lib/ci/api/api.rb | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/lib/ci/api/api.rb b/lib/ci/api/api.rb index 17bb99a2ae5..a6b9beecded 100644 --- a/lib/ci/api/api.rb +++ b/lib/ci/api/api.rb @@ -9,22 +9,14 @@ module Ci end rescue_from :all do |exception| - # lifted from https://github.com/rails/rails/blob/master/actionpack/lib/action_dispatch/middleware/debug_exceptions.rb#L60 - # why is this not wrapped in something reusable? - trace = exception.backtrace - - message = "\n#{exception.class} (#{exception.message}):\n" - message << exception.annoted_source_code.to_s if exception.respond_to?(:annoted_source_code) - message << " " << trace.join("\n ") - - API.logger.add Logger::FATAL, message - rack_response({ 'message' => '500 Internal Server Error' }, 500) + handle_api_exception(exception) end content_type :txt, 'text/plain' content_type :json, 'application/json' format :json + helpers ::SentryHelper helpers ::Ci::API::Helpers helpers ::API::Helpers helpers Gitlab::CurrentSettings |