-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
logging-sentry: Add config option for "server_name" #13412
Labels
Milestone
Comments
@medvedev are you interested in contributing this feature? It should be easy enough. |
Ah ah, I see you're working on it :). |
medvedev
added a commit
to medvedev/quarkus
that referenced
this issue
Nov 21, 2020
medvedev
added a commit
to medvedev/quarkus
that referenced
this issue
Nov 22, 2020
I m interested to work on this issue. How to assign issue to myself? |
@Erfankam the PR is already merged. But you can pick other Sentry's options from |
Closed by #13414 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Target extension: logging-sentry
Currently all the events in sentry have "server_name:localhost". Probably, it's a default value.
Proposal is to add a config option for sentry's server_name and pass it with every event.
Implementation ideas
Add config option
log.sentry.server-name
and use it forDefaultSentryClientFactory.SERVERNAME_OPTION
in iSentryConfigProvider.getProperty(String key)The text was updated successfully, but these errors were encountered: