Can NOT capture error wrapped rxjs.Observable.error()

[sizer]

• Review the documentation: https://docs.sentry.io/
• Search for existing issues: https://github.com/getsentry/sentry-javascript/issues
• Use the latest release: https://github.com/getsentry/sentry-javascript/releases
• Provide a link to the affected event from your Sentry account

Package + Version

• @sentry/browser
• @sentry/node
• raven-js
• raven-node (raven for node)
• other:

Version:

5.15.4

Description

My application(Angular@9) throws rxjs.Observable.error() and catch that by Sentry.captureException();.
At Sentry.io, I saw

Hub.push../node_modules/@sentry/hub/esm/hub.js.Hub.withScope
error
Non-Error exception captured with keys: ...

https://sentry.io/organizations/{myorg}/issues/1582573825/

It may be better that Sentry.captureException() could be to catch rxjs.Observable.error() simply.
Or had I mistaken for using @sentry/browser ??

code

import { Component } from "@angular/core";
import { Observable } from "rxjs";

@Component({
  selector: "app-root",
  template: `<button (click)="myProcess()"></button>`
})
export class AppComponent {
  public myProcess(): void {
    new Observable<{message: string}>(observer => {
      observer.error({message: "my failure..."});
    }).subscribe(result => console.log(result));
  }
}

@Injectable()
export class SentryErrorHandler implements ErrorHandler {
  constructor() {}
  handleError(error) {
    Sentry.captureException(error.originalError || error);
  }
}

[goosmurf]

I also hit this error but the OP wasn't super clear to me so I've created a repro: https://github.com/goosmurf/sentry-javascript-2533

You'll need to replace the DSN, then ng serve and visit https://localhost:4200/

Note that the handleError() is basically a no-op but an exception is still logged.

If you set a breakpoint against Sentry.captureException() you can further observe that the exception is logged to sentry.io prior to handleError() being called.

[pranav-js]

I use

import { throwError } from 'rxjs';

throwError(err);

Works fine, But these are captured 2 times in sentry

one with GlobalErrorHandlerService.handleError
other with Hub.push.lW6c.Hub.withScope

[github-actions]

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!

---

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

[github-actions]

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!

---

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

[pomali]

for those wondering - I solved similar problem using error property in Observer - like this

observable.pipe(...).subscribe({
  next: result => console.log(result),
  error: err => { /* handle or ignore error */}
})

this way default angular/rxjs error handler doesn't raise the exception