Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ERROR [sentry-rollup-plugin] Info: Successfully uploaded source maps to Sentry #14879

Closed
3 tasks done
heriec opened this issue Jan 1, 2025 · 5 comments · Fixed by getsentry/sentry-javascript-bundler-plugins#652
Assignees
Labels
Package: nuxt Issues related to the Sentry Nuxt SDK

Comments

@heriec
Copy link

heriec commented Jan 1, 2025

Is there an existing issue for this?

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/nuxt

SDK Version

^8.47.0

Framework Version

"nuxt": "^3.14.1592",

Link to Sentry event

No response

Reproduction Example/SDK Setup

I was using Sentry in a Nuxt project, and previously with Vue in Vite, I didn't encounter this issue.
The sentry-rollup-plugin reports an ERROR, but the output type is `Info.
And, in fact, there are no bugs when it is running.
like:

 ERROR  [sentry-rollup-plugin] Info: Using environment variables configured in ".env.sentry-build-plugin".
 ERROR  [sentry-rollup-plugin] Info: Sending telemetry data on issues and performance to Sentry. To disable telemetry, set options.telemetry to false.
 ERROR  [sentry-rollup-plugin] Info: Successfully uploaded source maps to Sentry

I checked the issues but didn't find similar cases, so it might be a problem on my end. I hope to get some help in resolving this.

  sentry: {
    sourceMapsUploadOptions: {
      org: 'xxxx',
      project: 'xxxxx',
      authToken: process.env.SENTRY_AUTH_TOKEN,
      sourcemaps: {
        filesToDeleteAfterUpload: ['./dist/**/*.js.map'],
      },
    },
  },
  sourcemap: { client: 'hidden' },

Steps to Reproduce

I'm sorry, I don't know how to provide code to reproduce my situation.
If possible, what would you like me to provide? For my previous setups in Vite, I followed the simplest instructions according to the official website.

Expected Result

without ERROR

Actual Result

Image

and
Image

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Jan 1, 2025
@github-actions github-actions bot added the Package: nuxt Issues related to the Sentry Nuxt SDK label Jan 1, 2025
@s1gr1d s1gr1d self-assigned this Jan 2, 2025
@s1gr1d
Copy link
Member

s1gr1d commented Jan 2, 2025

Hello, thanks for reaching out!

All logs were directed to stderr to not pollute stdout (someone reported an issue with knip).

But I see that this is not ideal, as it's confusing to see an ERROR message, when it's just an info. We'll discuss how to continue with that. My ideas for this:

  1. Direct info logs to stdout again and add an option to silence those logs or to redirect them to stderr (for tools like knip)
  2. Direct all logs to stdout and add an option to redirect all logs to stderr (also for tools like knip)

cc @lforst

@lforst
Copy link
Member

lforst commented Jan 7, 2025

Imo just because something prints to stderr it doesn't necessarily mean it is an error. I don't think we should change this, at least in the bundler plugin layer. Is there a reason why these messages are prefixed with an ERROR?

@s1gr1d
Copy link
Member

s1gr1d commented Jan 8, 2025

Probably Vite, Rollup, zsh or bash prefix stderr or console.error with ERROR. But so far I couldn't find a source.

When I node -e "console.error('test message')", the ERROR is not added, so I guess it's probably not coming from the terminal.

@kaiyoma
Copy link

kaiyoma commented Jan 10, 2025

I just reported something similar: getsentry/sentry-javascript-bundler-plugins#650

I think it's due to this change, which seems suspect: getsentry/sentry-javascript-bundler-plugins#624

@s1gr1d
Copy link
Member

s1gr1d commented Jan 13, 2025

I suspect Nuxt is changing the log output when something is logged as error: https://github.com/nuxt/nuxt/blob/main/packages/vite/src/utils/logger.ts#L27

We'll change this on our side to use .info, .debug etc. in this PR: getsentry/sentry-javascript-bundler-plugins#652

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Package: nuxt Issues related to the Sentry Nuxt SDK
Projects
Archived in project
4 participants