Support logging to Sentry

[ia3andy]

Fixes #3985

[geoand]

The test failures seem irrelevant and might be due to artifact caching.

Can you please rebase onto the latest master?

[ia3andy]

Ok then ;-)

…

On Mon, Dec 2, 2019 at 1:33 PM David M. Lloyd ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In extensions/logging-sentry/runtime/src/main/java/io/quarkus/sentry/SentryConfig.java <#5881 (comment)>: > + */ + @configitem(defaultValue = "WARNING") + public Level level; + + /** + * Sentry differentiates stack frames that are directly related to your application (“in application”) from stack frames + * that come from other packages such as the standard library, frameworks, or other dependencies. The difference is visible + * in the Sentry web interface where only the “in application” frames are displayed by default. + * + * You can configure which package prefixes your application uses with this option, which takes a comma separated list. + * + * This option is highly recommended as it affects stacktrace grouping and display on Sentry. See documentation: + * https://docs.sentry.io/clients/java/config/#in-application-stack-frames + */ + @configitem + public Optional<String> packages; I think it's important for all of our lists to be formatted the same way whenever possible. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#5881?email_source=notifications&email_token=AAQ664FDFYZYAXNXIF527KDQWT57ZA5CNFSM4JTAXHIKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCNREABI#discussion_r352570785>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAQ664ECOKGSE6MDYOW4XVDQWT57ZANCNFSM4JTAXHIA> .

-- Andy Damevin

[ia3andy]

I need to add something to the doc (adding WIP status to not merge)

[ia3andy]

Ok, I've done my changes