Incorrect injection of builders in AWS service classes
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 48/100
Línea de trabajo
Lee CloudWatchService.java y las clases de servicio correspondientes de IAM y Kinesis; después, inspecciona los registros de builders en IntegrationsModule.java. Traza cómo se crea cada cliente y confirma que cada creación obtiene un builder nuevo en lugar de reutilizar una instancia inyectada. La tarea está terminada cuando los tres builders de AWS siguen el mismo ciclo de vida seguro sin cambiar el comportamiento de configuración de los clientes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
It appears that the injection of the IamClientBuilder, KinesisClientBuilder and CloudWatchLogsClientBuilder in Integrations AWS Service classes is currently being done incorrectly. In the current implementation, a single instance of the builders is injected and reused for each client creation. I suppose this has no actual symptoms (unless there is some edge case I am not seeing), since the config properties are re-written each time the client is instantiated.
See example in CloudWatchService
See here for the registration of the builders.
I suggest that the change the constructors to instead inject a Provider<> or factory class instead that creates a new builder each time one is needed instead of reusing the same one.
- Lenguaje dominante
- Java
- Estrellas
- 17
- Forks
- 13
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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 Graylog2/graylog-plugin-integrations
-
bug triaged
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
Graylog2/graylog-plugin-integrations#1350 · 3 comentarios ·
-
Wrong link in README.mdAbiertobug triaged
Dificultad 1/5 Menos de una hora Aptitud para principiantes 45/100
Graylog2/graylog-plugin-integrations#1339 · 2 reacciones ·
-
bug triaged
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
Graylog2/graylog-plugin-integrations#1338 · 3 comentarios ·
-
Pager Duty Additional fieldsAbiertofeature triaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
Graylog2/graylog-plugin-integrations#1157 · 2 comentarios ·
-
Update GreyNoise Community IP Lookup AdapterQuizá libre de nuevo @roberto-graylog la tomó hace 1730 días y no hay ningún pull request abierto. Abiertofeature
Graylog2/graylog-plugin-integrations#957 · 1 asignado ·
Todos los issues de Graylog2/graylog-plugin-integrations
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
apache/arrow-java#1311 ·
Los mantenedores suelen responder en 2 días
-
bug triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
security
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
IBM/networking-java-sdk#204 ·