Check-ins not working

[simplenotezy]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/browser

SDK Version

7.51.2

Framework Version

Nest.js 9.3.9

Link to Sentry event

https://doubble.sentry.io/crons/sync-amplitude/?environment=production&project=4505119808946176

SDK Setup

Sentry.init({
	integrations: [
		new RewriteFrames({
			root: global.__rootdir__,
		}),
	],
	dsn: process.env.SENTRY_DSN,
	serverName:
		process.env.HOST_REGION || process.env.HOSTNAME
			? `${process.env.HOST_REGION ? `${process.env.HOST_REGION}_` : ''}${process.env.HOSTNAME || 'unknown'}`
			: undefined,
	release: process.env.APP_VERSION,
	environment: process.env.NODE_ENV, // XXX-production
	shutdownTimeout: 10000,
});

Steps to Reproduce

Basically I follow the documentation:

    this.monitorCheckInId = Sentry.captureCheckIn({
      monitorSlug: this.monitorSlug,
      status: status as any,
      ...(this.monitorCheckInId && {
        checkInId: this.monitorCheckInId,
      }),
    });

If I inspect the crons running I see:

[Nest] 1  - 05/11/2023, 9:15:30 AM     LOG [CRON] Will check in with Sentry: in_progress for slug YYYYY...
[Nest] 1  - 05/11/2023, 9:15:30 AM     LOG [CRON] Refreshing XXXX view...
[Nest] 1  - 05/11/2023, 9:18:19 AM     LOG [CRON] Done
[Nest] 1  - 05/11/2023, 9:18:19 AM     LOG [CRON] Will check in with Sentry: ok for slug YYYYY (b3eb88439b864bb8b72ee6d358195789)...

Heres an example of what I did before, and how I refactored the code to use the built-in feature:

enum SentryJobStatus {
   JobStarted = 'in_progress',
@@ -128,7 +126,9 @@ export abstract class BaseCommand extends CommandRunner {
   public async notifyJobCompleted(): Promise<void> {
     await this.jobCheckin(SentryJobStatus.JobCompleted);

-    Sentry.setContext('completed-job', {});
+    Sentry.configureScope((scope) => {
+      scope.setContext('completed-job', {});
+    });
   }

   private async jobCheckin(status: SentryJobStatus, isRunning = true) {
@@ -142,43 +142,30 @@ export abstract class BaseCommand extends CommandRunner {

-    Sentry.setContext('monitor', {
-      slug: this.monitorSlug,
+    Sentry.configureScope((scope) => {
+      scope.setContext('monitor', {
+        slug: this.monitorSlug,
+      });
     });

-    const endpointUrl = `https://sentry.io/api/0/organizations/XXXX/monitors/${this.monitorSlug}/checkins${
-      isRunning ? `/${this.monitorCheckInId ? this.monitorCheckInId : `/latest`}` : ''
-    }/`;
-
-    this.log(`Will check in with Sentry: ${status}...`);
-    await fetch(endpointUrl, {
-      method: !isRunning ? 'POST' : 'PUT',
-      headers: {
-        Authorization: `DSN ${dsn}`,
-        'Content-Type': 'application/json',
-      },
-      body: JSON.stringify({ status }),
-    })
-      .then(async (response) => {
-        if (!response.ok) {
-          return response.text().then((body) => {
-            throw new Error(`Request failed with status ${response.status}: ${body}`);
-          });
-        }
-        return response.json();
-      })
-      .then((data) => {
-        this.log(`Checked in with Sentry: ${data.id}`);
-        this.monitorCheckInId = data.id;
-      })
-      .catch((error) => {
-        this.error(`Job monitoring HTTP error. Status: ${error}`);
-      });
+    this.log(
+      `Will check in with Sentry: ${status} for slug ${this.monitorSlug}${
+        this.monitorCheckInId ? ` (${this.monitorCheckInId})` : ''
+      }...`,
+    );
+
+    this.monitorCheckInId = Sentry.captureCheckIn({
+      monitorSlug: this.monitorSlug,
+      status: status as any,
+      ...(isRunning && {
+        checkInId: this.monitorCheckInId,
+      }),
+    });
   }

Expected Result

It should report OK in the Cron status console, because they are all running as expected

Actual Result

They all report "MISSED"

It worked fine when we weren't using the Sentry SDK but just sending normal http requests.

[AbhiPrasad]

Hey @simplenotezy, thanks for writing in! It's not good that the checkins are not sending. Could you try enabling debug: true for Sentry via:

Sentry.init({
  debug: true,
});

and checking if there are any Sentry debug logs about sending checkins?

I'm going to try to repro w/ and example nest.js app myself, but if you get the chance to provide a reproduction would be amazing for use to debug further!

[teslitsky]

Had this issue too after begin using Sentry SDK for check-ins.
With debug: true Sentry logs the following lines:

Sentry Logger [log]: Integration installed: InboundFilters
Sentry Logger [log]: Integration installed: FunctionToString
Sentry Logger [log]: Integration installed: Console
Sentry Logger [log]: Integration installed: Http
Sentry Logger [log]: Integration installed: Undici
Sentry Logger [log]: Integration installed: OnUncaughtException
Sentry Logger [log]: Integration installed: OnUnhandledRejection
Sentry Logger [log]: Integration installed: ContextLines
Sentry Logger [log]: Integration installed: LocalVariables
Sentry Logger [log]: Integration installed: Context
Sentry Logger [log]: Integration installed: Modules
Sentry Logger [log]: Integration installed: RequestData
Sentry Logger [log]: Integration installed: LinkedErrors

[AbhiPrasad]

Hey folks, we've identified this bug - will fix with the next release. Sorry for the trouble!

[AbhiPrasad]

Hey we've released https://github.com/getsentry/sentry-javascript/releases/tag/7.52.0 with this being fixed. Could you guys upgrade and give it a try? Thanks!

[simplenotezy]

@AbhiPrasad Don't think it has worked. I tried locally but all jobs are still marked as 'missing'. I have pushed to production and will see how it goes over the next hours though, but don't think it's working