Remove default values for important fields such as logins
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 68/100
Direzione di ricerca
Inizia da src/main/java/org/ohdsi/webapi/FlywayConfig.java, intorno alla riga 73, e segui gli import di configurazione correlati per identificare dove vengono applicati i valori predefiniti. Controlla come le configurazioni di esempio forniscono questi valori obbligatori. Il lavoro è completato quando le chiavi obbligatorie mancanti causano un errore esplicito invece di usare silenziosamente valori hardcoded.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- Java
- Stelle
- 153
- Fork
- 187
- Merge medio
- 2g 12h
- PR unite (30g)
- 13
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di OHDSI/WebAPI
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 1 giorno
-
CDM cache warming still fails when cache.achilles.usePersonCount=false (missing `cc` alias in getBatchConceptRecordCount.sql)Forse già presa @DG47 l’ha presa 3 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
GET /conceptset/{id}/expression returns items in nondeterministic order (query has no ORDER BY)Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 67/100
OHDSI/WebAPI#2543 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
OHDSI/WebAPI#2513 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di OHDSI/WebAPI
Issue simili
-
[GLOBAL] Advancement resetAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
Vakore/ZappierGames#184 ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
lugassawan/swe-workbench#823 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
cassiancc/Raspberry-Core#69 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
IronCoreLabs/ironcore-alloy#351 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 79/100
checkstyle/checkstyle#22145 ·
I maintainer di solito rispondono entro 1 giorno