Update sentry.filter.ts #6
Conversation
| } | ||
| this.sentryService.captureException(exception); | ||
| this.sentryService.captureException(exception,true); | ||
| } |
There was a problem hiding this comment.
Issue: Additional parameter in function call
Reason: captureException is called with an additional boolean parameter that the method does not accept.
Affected dependencies:
• packages/sentry-plugin/src/sentry.service.ts - captureException: Method does not accept additional parameter
Suggestion: Update captureException method to handle the new boolean parameter or remove it from the call.
Required changes in dependencies:
• packages/sentry-plugin/src/sentry.service.ts - captureException: Modify captureException to accept and handle the boolean parameter
There was a problem hiding this comment.
It seems there was a misunderstanding. The issue is not with the captureException method in sentry.service.ts, but rather with the Sentry SDK itself, which does not support this additional flag parameter. We need to ensure that either the Sentry SDK version supports this change or update the code to align with the SDK's API.
There was a problem hiding this comment.
It seems there was a misunderstanding. The captureException function now includes an additional parameter flag. All calls to this function need to be updated accordingly in the affected files, not just modifying the method signature. Please ensure that all instances where captureException is called include the new flag parameter with an appropriate value.
Description
Please include a summary of the changes and the related issue.
Breaking changes
Does this PR include any breaking changes we should be aware of?
Screenshots
You can add screenshots here if applicable.
Checklist
📌 Always:
👍 Most of the time: