cube-js/cube

Sentry integration

Open

#6,249 opened on Mar 6, 2023

View on GitHub
 (2 comments) (6 reactions) (0 assignees)Rust (1,965 forks)batch import
help wanted

Repository metrics

Stars
 (19,563 stars)
PR merge metrics
 (Avg merge 5d 16h) (138 merged PRs in 30d)

Description

Sentry integration

Using CubjeJS in production requires us to monitor the errors that are thrown in this environment. For this we use the common Sentry tool. Unfortunately, we can't make it work in CubeJS as of today.

We tried using express error handlers defined in the initApp parameter of the CubeJS config, but it doesn't seem to catch errors yet.

Would it be possible to either provide support for Sentry or other monitoring tools out of the box, or else give more control over the express Response object to add custom error handlers for this purpose ?

Contributor guide

Research direction
Research how to add custom error handling middleware in CubeJS via initApp or other hooks, and how to integrate Sentry's request error handler (e.g., Sentry.Handlers.errorHandler). Look at existing Express error handling documentation and CubeJS's plugin or middleware hooks.
Tech stack
javascripttypescriptnodejsexpress
Domain
backendobservabilityapi
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
JavaScriptNode.jsExpress
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75