use getenv in configs
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 25/100
- Type d'issue
- Fonctionnalité
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- docker, shell
- Domaine
- devops, infrastructure
Piste de recherche
Commencez par localiser la configuration de l’image Docker et l’initialisation existante des variables d’environnement qui écrit des valeurs dans config.php. Identifiez quels paramètres nécessaires au fonctionnement continu sont encore codés en dur et évaluez si getenv peut les fournir de manière sûre, y compris dans les cas de secrets de conteneur et de rotation. Le travail est terminé lorsque les paramètres concernés lisent systématiquement les variables d’environnement sans supprimer la prise en charge de la configuration manuelle dans config.php.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
How to use GitHub
- Please use the 👍 reaction to show that you are interested into the same feature.
- Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
- Subscribe to receive notifications on status change and new comments.
Is your feature request related to a problem? Please describe.
Simply put, I can understand using the environment variables for initialising NC, but then not using environment variables for continued operations seems silly to me. Why hardcode the values when they are initially read from environment variables anyway?
Additionally using environment variables instead of hardcoding would allow using container secrets directly.
Additionally rotating DB passwords or locations becomes a breeze since it would all be done through environment variables.
Describe the solution you'd like
Star using: 'var' => getenv('ENV_VAR') as much as possible since manually editing config.php can be done eitherway.
Describe alternatives you've considered
Doing it myself, while yes I've done this. I see that this would be likely to benefit other people as well.
Additional context
Since this is an "advanced" image when compared to AIO I can understand potential hesitancy to adopt my suggestions. Yet I believe that it will help since it doesn't detract from original deployment methods in any way.
- Langage dominant
- Shell
- Étoiles
- 7.4k
- Forks
- 1.9k
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de nextcloud/docker
-
1. To develop bug feature: cron image
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Drop paragarph about anonymous (unnamed) volumes under Persistent Storage section fo the docsOuvertebug docs
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
-
0. Needs triage bug
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
-
Imagick architecture mismatch (x86_64 vs aarch64) on official arm64 image causes Apache segfaultsOuverte0. Needs triage bug
Difficulté 4/5 3-5 jours Accessibilité débutants 50/100
Toutes les issues de nextcloud/docker
Issues similaires
-
agent/quality help wanted origin/quality quality stage/010-actionable testing
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
hivecommons/hotshot#227 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
sleeyax/promptfiles#20 ·
-
Add PhotocraftOuverteenhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
Les mainteneurs répondent en général sous 1 jour
-
Missing icon: novelWriterOuverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 78/100
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100