Replace field injection with setter injection in `3.x`
Los mantenedores suelen responder en 1 día
@jaykataria1111 ya está trabajando en esto.
Desde el 3/11/2024.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
As issue #2766 shows, there are still some Log4j plugin builders that don't have setters for all their configuration attributes.
Since field injection becomes more problematic in newer Java version, I believe we should add the missing setters and make sure all new attributes come with a public setter.
Therefore I propose:
- to modify the annotation processor in
2.xto fail if a plugin builder attribute does not have a public setter (or at least a wither). - fix the compilation errors in
2.x.
Regarding 3.x I would prefer to remove field injection completely from log4j-plugins and use builders and setter injection everywhere. IMHO our DI subsystem does not need to have all the features of a fully-fledged DI.
Obviously we might still need field injection if we want to support all the existing 2.x plugins, but I would still love a 200 KiB dependency injection system.
- Lenguaje dominante
- Java
- Estrellas
- 3.6k
- Forks
- 1.8k
- Merge medio
- 5 d 6 h
- PR fusionados (30 d)
- 24
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de apache/logging-log4j2
-
additivity attributeAbiertowaiting-for-maintainer
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
apache/logging-log4j2#4475 ·
Los mantenedores suelen responder en 1 día
-
Migrate `log4j-api-test` tests to JUnit 5Quizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abiertoapi priority: P3 tests waiting-for-maintainer
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
apache/logging-log4j2#4430 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
`Rfc5424Layout` does not escape the `newLineEscape` replacement in SD-PARAM-VALUEPosiblemente ocupada @lunarbae628 la tomó hace 5 días. Abiertobug layouts priority: P3
apache/logging-log4j2#4464 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
configuration enhancement layouts
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
apache/logging-log4j2#4463 ·
Los mantenedores suelen responder en 1 día
-
`evaluateOnStartup="true"` saves logs under wrong file names after missing scheduled rollQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abiertoappenders:Rolling bug priority: P1 waiting-for-maintainer
Dificultad 3/5 1-2 días Aptitud para principiantes 30/100
apache/logging-log4j2#4449 ·
Los mantenedores suelen responder en 1 día
Todos los issues de apache/logging-log4j2
Issues similares
-
untriaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
opensearch-project/opensearch-prometheus-exporter#680 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Los mantenedores suelen responder en 2 días
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
-
Good First Issue
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
hiero-ledger/hiero-sdk-java#3059 ·
Los mantenedores suelen responder en 1 día
-
bug live-tv platform:desktop platform:mobile platform:tv
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
fk0u/cleanyourtv-tahoe#102 ·
Los mantenedores suelen responder en 1 día