Skip to content

Missing setter on GelfHandler for multiple additional fields #125

Open
@goedtim

Description

@goedtim

We are using the GelfHandler as part of a JBoss EAP6 application and create/configure it as a custom handler in the JBoss configuration. Furthermore we need to provide multiple additional fields.

Current issue is, that JBoss inspects the Handler class and for any property provided in the configuration, it expects a matching setter. Example property graylogHost needs a setter setGraylogHost().

For all supported properties there is a matching setter, also for a single additional field, but not for multiple additional fields. Also, JBoss configuration does not allow repeating the same property multiple times. So we would need a setter setAdditionalFields(String fields) with a parsing logic similar to what is currently done in the constructor.

I could provide a fix, just want to discuss here if that's fine or if I am on a wrong track. Also, if I provide a fix, how long would it take for a new release as we would probably not install custom build software in our PROD environment?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions