Remove default values for important fields such as logins
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
Línea de trabajo
Empieza en src/main/java/org/ohdsi/webapi/FlywayConfig.java, alrededor de la línea 73, y sigue las importaciones de configuración relacionadas para identificar dónde se aplican los valores predeterminados. Comprueba cómo las configuraciones de ejemplo proporcionan estos valores obligatorios. Se considera terminado cuando la ausencia de claves obligatorias provoca un error claro en lugar de usar silenciosamente valores codificados de forma fija.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Currently the configuration process sets some default values if the required keys do not exist in the config. While a useful crutch, I think it would be much more useful to simply loudly fail instead of falling back to hardcoded values.
The rationale is that if I do not use the default values from the example (e.g. login and password for the postgres database), I'd much prefer to have it error out early than to dig through the logs wondering why is it using a different login than what I put in the config file for some reason. As the sample configs need to be changed anyway, hardcoding default values is more harmful than helpful currently.
I think this comes from this file and related imports:
https://github.com/OHDSI/WebAPI/blob/master/src/main/java/org/ohdsi/webapi/FlywayConfig.java#L73
- Lenguaje dominante
- Java
- Estrellas
- 153
- Forks
- 186
- Merge medio
- 2 d 12 h
- PR fusionados (30 d)
- 13
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin 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 OHDSI/WebAPI
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
CDM cache warming still fails when cache.achilles.usePersonCount=false (missing `cc` alias in getBatchConceptRecordCount.sql)Posiblemente ocupada @DG47 la tomó hace 2 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
GET /conceptset/{id}/expression returns items in nondeterministic order (query has no ORDER BY)Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 67/100
OHDSI/WebAPI#2543 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
OHDSI/WebAPI#2513 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de OHDSI/WebAPI
Issues similares
-
Optimize consumer version heatmap aggregation to avoid repeated scansPosiblemente ocupada @praykaria31 la tomó hoy. Abiertotype/enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Apicurio/apicurio-registry#10662 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
a:false-positive
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 1 día
-
status: waiting-for-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
spring-projects/spring-boot#52347 ·
Los mantenedores suelen responder en 2 días
-
bug untriaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
opensearch-project/opensearch-migrations#3480 ·
Los mantenedores suelen responder en 2 días
-
enhancement feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
rsp777/service-pulse-app#28 ·