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

[Feature Request] Option to disable trace logs in aQute.remote.util.Link<L, R> #5240

Closed
amitjoy opened this issue May 6, 2022 · 6 comments
Closed
Assignees
Milestone

Comments

@amitjoy
Copy link
Member

amitjoy commented May 6, 2022

External applications could make use of Bnd Remote Agent to connect to remote runtimes and the trace logs appear consistently and pollutes the remote runtime console heavily. It would be better to introduce a system property to disable the logging on demand.

@bjhargrave
Copy link
Member

The logging is SLF4J logging so you have complete control over the log output. When using the slf4j-simple binding, you can use the system property -Dorg.slf4j.simpleLogger.log.aQute.lib.link.Link=error to only display the error level log output. If using a different slf4j binding, you will need to configure the log level according to the binding's documentation.

@bjhargrave
Copy link
Member

When launching Bndtools for testing/debugging, we use the logback slf4j binding with this configuration file to control the log levels for various loggers.

@amitjoy
Copy link
Member Author

amitjoy commented May 6, 2022

I think we need to introduce logging support here

@bjhargrave
Copy link
Member

I think we need to introduce logging support here

In the title of this issue you named aQute.lib.link.Link which is, of course, where I looked. Now you seem to instead be talking about aQute.remote.util.Link which is a different class. Please correct the title to refer to the class you are talking about. Thanks.

@amitjoy
Copy link
Member Author

amitjoy commented May 6, 2022

Yeah, you are right. It was confusing. My bad!! I will change it right away.

@amitjoy amitjoy changed the title [Feature Request] Option to disable trace logs in aQute.lib.link.Link<L, R> [Feature Request] Option to disable trace logs in aQute.remote.util.Link<L, R> May 6, 2022
@bjhargrave bjhargrave added this to the 6.4 milestone Jul 8, 2022
@pkriens
Copy link
Member

pkriens commented Aug 19, 2022

Closed by #5345

@pkriens pkriens closed this as completed Aug 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants