Skip to content

Provide SLF4J bridge for logging #15122

Open
@hendrikebbers

Description

🆕🐥 First Timers Only

This issue is reserved for people who have never contributed to Hedera or any open source project in general.
We know that creating a pull request (PR) is a major barrier for new contributors.
The goal of this issue and all other issues labeled by 'good first issue' is to help you make your first contribution to Hedera.

👾 Description of the issue

The project has an internal logging framework that is defined by the swirlds-logging module. The source of the module that can be found here. A general documentation of the framework can be found here.

We decided to create our custom logging implementation since it is faster for our use cases than other frameworks and does not have the complexity of log4j2.

Since most of our dependencies use slf4j or log4j2 we decided to provide extensions for the logging framework that forward log4j2 and slf4j logging messages directly to our logging implementation. The module swirlds-logging-log4j-appender that can be found here already forwards all log4j logging to our logging system.

We need to do the same for slf4j.

Proposed Solution

The new module swirlds-logging-slf4j-appender needs to be created and structured similarly to swirlds-logging-log4j-appender. The module must provide a solution that forwards all slf4j log messages to our system. Like the given module, the new one must use Java SPI to load the extension automatically once it is on the classpath.

Needed skillset

While this issue is a good first issue that does not require any deep knowledge of Hedera and the internals of this repo, fixing it requires a lot of Java knowledge.o not recommend this issue for developers who Therefore, we d are new to Java.

📋 Step by step guide to do a contribution

If you have never contributed to an open source project at GitHub, the following step-by-step guide will introduce you to the workflow. More information and concrete samples for shell commands for each step can be found in our CONTRIBUTING.md file.
A more detailed general documentation of the GitHub PR workflow can be found here.

  • Claim this issue: Comment below that you are interested in working on the issue
  • Wait for assignment: A community member with the given rights will add you as an assignee of the issue
  • Fork the repository: You can do that in GitHub (by simply clicking the 'fork' button).
  • Check out the forked repository
  • Create a feature branch for the issue. We do not have a hard naming definition for branches but it is best practice to prefix the branch name with the issue id.
  • Solve the issue in your branch.
  • Commit your changes: Here, it is needed to add sign-off information to the commit to accept the "Developer Certificate of Origin" (https://developercertificate.org). More details can be found in our CONTRIBUTING.md
  • Start a Pull Request (PR): We have a pattern for naming pull requests that a GitHub Action checks. We use that pattern to support the creation of automatic release notes.
  • Check GitHub Actions: Several GitHub Actions will be triggered automatically for each PR. If a GitHub Action fails and you do not understand the cause of that error do not hesitate to add a comment to the PR and ask the Hedera developer community for support.
  • Wait for reviews: Members of the Hedera developer community will review your PR. If a reviewer finds any missing pieces or a problem, he or she will start a discussion with you and describe the next steps for solving the problem.
  • You did it 🎉: We will merge the fix in the develop branch. Thanks for being part of the Hedera community as an open-source contributor ❤️

🎉 Contribute to Hacktoberfest

Solve this issue as part of the Hacktoberfest event and get a chance to receive cool goodies like a T-Shirt. 🎽

🤔 Additional Information

If you have any questions, just ask us directly in this issue by adding a comment. You can join our community chat at Discord. A general manual about open-source contributions can be found here.

Metadata

Assignees

Labels

Base LoggingTickets pertaining to the base logging libraryGood First IssueEither a small change or a medium change concentrated in a single component.ImprovementCode changes driven by non business requirements.JavaPull requests that update Java codeModularizationIssues or PRs related to modularizationPlatformTickets pertaining to the platformhacktoberfestIssues shown by lists for the Hacktoberfest and made for newcomers to do the first contribution.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions