Skip to content
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

Make logger for jar file add-ons configurable in UI #3921

Merged
merged 1 commit into from
Dec 16, 2023

Conversation

mherwege
Copy link
Contributor

Add-ons provided as jar files in the addons in the add-on folder did not have any information about loggers, so the logger could not be set through the UI.
As jar files are often provided to test new add-on functionality, the ability to set logging in the UI is relevant.
This PR adds the symbolic name as the name for the logger, which should be the main logger for most of the jar file provided add-ons.

Signed-off-by: Mark Herwege <mark.herwege@telenet.be>
Copy link
Member

@kaikreuzer kaikreuzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@kaikreuzer kaikreuzer merged commit d3ebc82 into openhab:main Dec 16, 2023
3 checks passed
@kaikreuzer kaikreuzer added the enhancement An enhancement or new feature of the Core label Dec 16, 2023
@kaikreuzer kaikreuzer added this to the 4.1 milestone Dec 16, 2023
@mherwege mherwege deleted the logger branch December 16, 2023 11:21
@mvalla
Copy link
Contributor

mvalla commented Dec 24, 2023

@mherwege if I install via Eclipse Dev Environment an addon, no logging configuration appears in MainUI for the addon.
Is there any change I should make to my addon in order to support MainUI-based logging configuration?
Or is it a limition/bug?

@wborn
Copy link
Member

wborn commented Dec 24, 2023

Or is it a limition/bug?

It depends on Karaf functionality which is unavailable when debugging in Eclipse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature of the Core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants