Add aspectjweaver Dependency for Sentry Cron Monitoring
Problem Statement
While implementing Sentry for cron job monitoring in a Kotlin/Spring application, I encountered an issue where the advisor was not being registered as a bean, which caused the monitoring to fail silently. This was due to the missing aspectjweaver dependency.
I propose the following solutions to prevent this issue for other users:
-
Update Documentation: Clearly state in the Sentry documentation that
org.aspectj:aspectjweaverneeds to be included as a dependency for proper operation of AOP-based features like cron job monitoring. -
Module Dependency: Include
org.aspectj:aspectjweaverdirectly within the Sentry module dependencies, ensuring it is automatically available when the Sentry module is used.
Solution Brainstorm
No response
I prefer the latter. Please review. thank you
Thanks for the report @flex-junkyu, we'll discuss internally and update here once we've made a decision on how to proceed.
Thanks for reporting this, we have decided to document it, as:
aspectjweavercould be incompatible with certain environments (e.g. GraalVM)- this is only needed for Crons, so adding it as a dependency would include it for everyone unnecessarily