-
-
Notifications
You must be signed in to change notification settings - Fork 584
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unterstützung von Umgebungsvariablen zur Konfiguration #405
Comments
Issue-Label Bot is automatically applying the label Links: app homepage, dashboard and code for this bot. |
@StefanSchoof AutomaticEnv ist bereits gesetzt. Bis auf den fehlenden Prefix sollte das also gehen. Mir ist allerdings nicht klar, wie das innerhalb von Listen wie z.B. den vehicles funktionieren soll? |
Aus https://github.com/spf13/viper#accessing-nested-keys. ( |
SO oder viper Ticket aufmachen? |
Ich probiere und Google noch ein bisschen und mache das dann. |
Das ist ein Problem in viper: spf13/viper#935. Insgesamt scheint es mit dem Unmarshal und env vars nicht so gut zu gehen: spf13/viper#761 Denke daher, das ist nichts was man einfach hinbekommt :( |
Schade :( |
Is your feature request related to a problem? Please describe.
Ich möchte gerne Passworter und andere Konfiguration soweit wie möglich trennen. (Berufskrankheit ;) ) Das ermöglicht mir meine Konfiguration einfach bei github hochzuladen (https://github.com/StefanSchoof/evcc-config) und somit kann ich einfach auf einen alten Config stand und habe schon ein Backup. (Nach Linus Torvalds: "Real men don’t use backups, they post their stuff on a public ftp server and let the rest of the world make copies." )
Describe the solution you'd like
Aktivieren Umgebungsvariablen in viper https://github.com/spf13/viper#working-with-environment-variables, so das man das Passwörter per Umgebungsvariable an evcc übergeben kann.
Describe alternatives you've considered
Platzhalter für Passwörter in der config, was zurück drehen der config und aktualisieren von gut schwieriger macht.
Additional context
The text was updated successfully, but these errors were encountered: