feat(api-service): Log API Service level in NewRelic #7739

SokratisVidros · 2025-02-17T15:47:32Z

What changed? Why was the change needed?

Allow us to correlate logs to customers faster. See also https://github.com/novuhq/packages-enterprise/pull/269
Update to latest Pino and Nestjs Pino
Fix issue with duplicate custom Props in each JSON Log entry. See customProps always logs duplicate keys pinojs/pino-http#216

netlify · 2025-02-17T15:48:01Z

✅ Deploy Preview for dev-web-novu ready!

Name	Link
🔨 Latest commit	`6f81260`
🔍 Latest deploy log	https://app.netlify.com/sites/dev-web-novu/deploys/67c08b935c2ba20008801510
😎 Deploy Preview	https://deploy-preview-7739.dashboard.novu-staging.co
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify · 2025-02-17T15:53:20Z

✅ Deploy Preview for dashboard-v2-novu-staging ready!

Name	Link
🔨 Latest commit	`6f81260`
🔍 Latest deploy log	https://app.netlify.com/sites/dashboard-v2-novu-staging/deploys/67c08b9359da6c00084b62fe
😎 Deploy Preview	https://deploy-preview-7739.dashboard-v2.novu-staging.co
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

jainpawan21 · 2025-02-17T17:08:26Z

libs/application-generic/src/logging/index.ts

@@ -120,6 +121,7 @@ export function createNestLoggingModuleOptions(settings: ILoggerSettings): Param
          userId: req?.user?._id || null,
          environmentId: req?.user?.environmentId || null,
          organizationId: req?.user?.organizationId || null,
+          apiServiceLevel: req?.user?.apiServiceLevel || null,


@SokratisVidros as per my knowledge apiServiceLevel is on organization level. not on user level. a user can be member of two org one free and other business

Never mind, I just checked EE PR. The confusion is now resolved.

pkg-pr-new · 2025-02-25T16:58:36Z

Open in Stackblitz

@novu/client

npm i https://pkg.pr.new/novuhq/novu/@novu/client@7739

@novu/headless

npm i https://pkg.pr.new/novuhq/novu/@novu/headless@7739

@novu/node

npm i https://pkg.pr.new/novuhq/novu/@novu/node@7739

@novu/notification-center

npm i https://pkg.pr.new/novuhq/novu/@novu/notification-center@7739

novu

npm i https://pkg.pr.new/novuhq/novu@7739

@novu/providers

npm i https://pkg.pr.new/novuhq/novu/@novu/providers@7739

@novu/shared

npm i https://pkg.pr.new/novuhq/novu/@novu/shared@7739

commit: 6f81260

Allow us to correlate logs to customers faster.

Bump Pino and fix duplicate "user" keys in the final JSON Log entry.

SokratisVidros requested a review from ChmaraX February 17, 2025 15:47

SokratisVidros temporarily deployed to Linting February 17, 2025 15:47 — with GitHub Actions Inactive

SokratisVidros requested review from scopsy and djabarovgeorge February 17, 2025 15:47

jainpawan21 reviewed Feb 17, 2025

View reviewed changes

jainpawan21 approved these changes Feb 17, 2025

View reviewed changes

SokratisVidros force-pushed the log_api_service_level branch from f5ea0d6 to 0769b2f Compare February 25, 2025 10:25

SokratisVidros temporarily deployed to Linting February 25, 2025 10:25 — with GitHub Actions Inactive

SokratisVidros force-pushed the log_api_service_level branch from 0769b2f to 3d54da4 Compare February 25, 2025 16:54

SokratisVidros temporarily deployed to Linting February 25, 2025 16:54 — with GitHub Actions Inactive

github-actions bot added the @novu/shared label Feb 25, 2025

SokratisVidros temporarily deployed to Linting February 25, 2025 17:50 — with GitHub Actions Inactive

SokratisVidros force-pushed the log_api_service_level branch from 3f71eb3 to c451c24 Compare February 26, 2025 14:26

SokratisVidros temporarily deployed to Linting February 26, 2025 14:26 — with GitHub Actions Inactive

SokratisVidros temporarily deployed to Linting February 26, 2025 14:27 — with GitHub Actions Inactive

SokratisVidros force-pushed the log_api_service_level branch from c451c24 to b752703 Compare February 26, 2025 21:18

SokratisVidros temporarily deployed to Linting February 26, 2025 21:18 — with GitHub Actions Inactive

SokratisVidros added 2 commits February 27, 2025 17:54

feat(api-service): Log API Service level in NewRelic

a22210d

Allow us to correlate logs to customers faster.

feat(application-generic): Bump Pino

6f81260

Bump Pino and fix duplicate "user" keys in the final JSON Log entry.

SokratisVidros force-pushed the log_api_service_level branch from b752703 to 6f81260 Compare February 27, 2025 15:58

SokratisVidros temporarily deployed to Linting February 27, 2025 15:58 — with GitHub Actions Inactive

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(api-service): Log API Service level in NewRelic #7739

feat(api-service): Log API Service level in NewRelic #7739

SokratisVidros commented Feb 17, 2025 •

edited

Loading

netlify bot commented Feb 17, 2025 •

edited

Loading

netlify bot commented Feb 17, 2025 •

edited

Loading

jainpawan21 Feb 17, 2025

jainpawan21 Feb 17, 2025

pkg-pr-new bot commented Feb 25, 2025 •

edited

Loading

feat(api-service): Log API Service level in NewRelic #7739

Are you sure you want to change the base?

feat(api-service): Log API Service level in NewRelic #7739

Conversation

SokratisVidros commented Feb 17, 2025 • edited Loading

What changed? Why was the change needed?

netlify bot commented Feb 17, 2025 • edited Loading

✅ Deploy Preview for dev-web-novu ready!

netlify bot commented Feb 17, 2025 • edited Loading

✅ Deploy Preview for dashboard-v2-novu-staging ready!

jainpawan21 Feb 17, 2025

Choose a reason for hiding this comment

jainpawan21 Feb 17, 2025

Choose a reason for hiding this comment

pkg-pr-new bot commented Feb 25, 2025 • edited Loading

SokratisVidros commented Feb 17, 2025 •

edited

Loading

netlify bot commented Feb 17, 2025 •

edited

Loading

netlify bot commented Feb 17, 2025 •

edited

Loading

pkg-pr-new bot commented Feb 25, 2025 •

edited

Loading