Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

Can't add environment variable that contains a comma #40

Closed
jrtilson opened this issue Mar 9, 2018 · 3 comments
Closed

Can't add environment variable that contains a comma #40

jrtilson opened this issue Mar 9, 2018 · 3 comments
Assignees
Labels
Milestone

Comments

@jrtilson
Copy link

jrtilson commented Mar 9, 2018

If we enter:

fargate service env set myapp --env MY_VAR=has,comma

The result is:

[!] Invalid environment variable
comma must be in the form of KEY=value

This is an issue because I'm trying to supply a mongodb string, ie:

MONGO_URL=mongodb://user:pass@host1,host2/db?auth=123&key=456

Can the value portion be escaped somehow?

@jpignata jpignata self-assigned this Mar 12, 2018
@jpignata jpignata added this to the v0.3.0 milestone Mar 12, 2018
@jpignata jpignata added the bug label Mar 12, 2018
@abarfoot
Copy link

Possible workaround: When the value is a url, try substituting %2c for the comma.

@holyjak
Copy link

holyjak commented May 23, 2019

It seems pretty easy to fix according to turnerlabs@f2fe902, replacing Flags().StringSliceVarP with Flags().StringArrayVarP. Could it be done? Please? 🙏

@PaulMaddox
Copy link
Contributor

Merged :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants