Skip to content
Snippets Groups Projects

Add Sentry logging to API calls

Merged Stan Hu requested to merge add-sentry-logging-to-api into master
All threads resolved!

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?

#21043 (closed)

Does this MR meet the acceptance criteria?

Merge request reports

Pipeline #4036255 passed

Pipeline passed for 170885ed on add-sentry-logging-to-api

Test coverage 90.39% (0.00%) from 1 job

Merged by avatar (Apr 10, 2025 9:16am UTC)

Loading

Pipeline #4038231 passed

Pipeline passed for 9ea01f32 on master

Test coverage 90.88% (0.00%) from 1 job

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Stan Hu Marked the task All builds are passing as completed

    Marked the task All builds are passing as completed

  • Stan Hu Resolved all discussions

    Resolved all discussions

  • Robert Speicher
  • Robert Speicher
  • Stan Hu Added 28 commits:

    Added 28 commits:

  • Stan Hu Added 1 commit:

    Added 1 commit:

    • 13eaefcf - Add Sentry logging to API calls
  • Stan Hu Added 1 commit:

    Added 1 commit:

  • Stan Hu Added 19 commits:

    Added 19 commits:

  • mentioned in issue #21410 (closed)

  • Robert Speicher Resolved all discussions

    Resolved all discussions

  • Robert Speicher Status changed to merged

    Status changed to merged

  • Robert Speicher mentioned in commit 9ea01f32

    mentioned in commit 9ea01f32

  • Stan Hu mentioned in issue #21276 (closed)

    mentioned in issue #21276 (closed)

  • Please register or sign in to reply
    Loading