Add Sentry logging to API calls
What does this MR do?
This MR adds support for Sentry logging in the API.
Are there points in the code the reviewer needs to double check?
Since the Grape::Middleware
doesn't have a params
method, I had to define one using the Rack Request.
Why was this MR needed?
We are missing a lot of useful errors in the API causing git push/pull errors
What are the relevant issue numbers?
Does this MR meet the acceptance criteria?
-
CHANGELOG entry added - Tests
-
Added for this feature/bug -
All builds are passing
-
-
Conform by the style guides -
Branch has no merge conflicts with master
(if you do - rebase it please) -
Squashed related commits together
Merge request reports
Activity
Added 112 commits:
-
c7320748...c5aa31c8 - 111 commits from branch
master
- 06e1b926 - Add Sentry logging to API calls
-
c7320748...c5aa31c8 - 111 commits from branch
Added 1 commit:
- a817273c - Add Sentry logging to API calls
Added 1 commit:
- 51372c11 - Filter out only abnormal API exceptions
Added 121 commits:
-
51372c11...57995172 - 120 commits from branch
master
- a3f1d28a - Add Sentry logging to API calls
-
51372c11...57995172 - 120 commits from branch
Added 245 commits:
-
a3f1d28a...148b8487 - 243 commits from branch
master
- a8d15435 - Add Sentry logging to API calls
- 6993c34d - Add spec for API helper
-
a3f1d28a...148b8487 - 243 commits from branch
Added 1 commit:
- af637569 - Move API Sentry logging to 8.12
Milestone changed to %8.12
Added backstage label
Reassigned to @rspeicher
Added 1 commit:
- 5c432c09 - Consolidate Sentry reporting code into beginning
Added 1 commit:
- 5f2a6507 - Add Sentry logging to API calls
Marked the task CHANGELOG entry added as completed
Marked the task Conform by the style guides as completed
Marked the task Squashed related commits together as completed
- Resolved by Stan Hu
- Resolved by Robert Speicher
- Resolved by Robert Speicher
- Resolved by Stan Hu
Added 28 commits:
-
5f2a6507...02591b04 - 27 commits from branch
master
- 1b473b86 - Add Sentry logging to API calls
-
5f2a6507...02591b04 - 27 commits from branch
Added 1 commit:
- 13eaefcf - Add Sentry logging to API calls
Added 1 commit:
- 1ca90f73 - Add Sentry logging to 8.12
Added 19 commits:
-
1ca90f73...7b0b2417 - 18 commits from branch
master
- 170885ed - Add Sentry logging to API calls
-
1ca90f73...7b0b2417 - 18 commits from branch
mentioned in issue #21410 (closed)
mentioned in commit 9ea01f32
mentioned in issue #21276 (closed)