-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
GH-394 Add Sentry integration #808
base: master
Are you sure you want to change the base?
Conversation
options.setBeforeSend((event, hint) -> { | ||
Throwable throwable = event.getThrowable(); | ||
|
||
if (throwable != null) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trochę mało wydajne te streamy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co proponujesz, zwykłe pętle for()
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jakieś hashmapy sety coś takiego...
...core/src/main/java/com/eternalcode/core/bridge/sentry/SentryLiteCommandExceptionHandler.java
Show resolved
Hide resolved
@@ -45,6 +46,7 @@ public static List<Class<?>> scanClasses(String packageToScan, ClassLoader class | |||
return loadedClasses; | |||
} | |||
catch (IOException | ClassNotFoundException exception) { | |||
Sentry.captureException(exception); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trochę łopatologiczne Lucky jakoś globalnie kiedyś łapał błędy ale nw czy to tylko na wątki działa
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
poprawiłem indentację
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ale łopatologiczne jest używanie wszędzie Sentry#captureException
, czy co masz na myśli? Musimy tak robić bo ja nie łapię wszystkich błędów celowo, np. tych argumentów z LC, bo przecież tam to jest wina stricte usera
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wątek w Javie ma coś takiego jak uncatched handler tyle, że musi wywalić wątek aby to złapało to część case już ogarnie... warto pomyśleć nad tym
Resolves #394