localstack / localstack/localstack-dagger-module
Accommodate comma-separated config values
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 3
- Forks
- 0
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
There's no way to provide comma-separated config values through the configuration argument.
For example, I want to specify a list of services via the SERVICES config variable (e.g., SERVICES=s3,ec2,secretsmanager,ecs,iam,sts,fis). However, the configuration parsing code naively splits the provided string on commas:
https://github.com/localstack/localstack-dagger-module/blob/6e2683e20e54e7ceab130ebe2abed050b6670ed2/src/localstack/main.py#L47-L52
As such, when specifying configuration as follows, ec2 is recognized as a separate config variable rather than another service in the list of services that comprise the value for the SERVICES config variable:
dag.Localstack().Start(dag.LocalstackStartOpts{
Configuration: "SERVICES=s3,ec2,secretsmanager,ecs,iam,sts,fis"
})
There are a few solutions to this:
- use a non-comma delimiter to distinguish configuration key-value pairs (e.g., use
;as the delimiter:"SERVICES=s3,ec2,secretsmanager;AWS_DEFAULT_REGION=east-us-1") - accommodate escaped quotes (e.g.,
"SERVICES=\"s3,ec2,secretsmanager\",AWS_DEFAULT_REGION=east-us-1")
Guide de contribution
Ouvrir 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.
Piste de recherche
Commencez par lire le code d’analyse de la configuration dans src/localstack/main.py aux lignes 47-52 et suivez la façon dont l’argument de configuration est divisé. Comparez les approches proposées pour le délimiteur et l’échappement, puis vérifiez les tests ou les commandes de test existants du dépôt. C’est terminé lorsque les valeurs séparées par des virgules restent incluses dans une seule valeur de configuration et que les entrées de configuration distinctes sont toujours analysées correctement.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- devops
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 38/100