Code-server: Add environment variable support for existing runtime flags

[Omeryl]

It was brought up in Discord by @PhilsLab that we should allow environment variables to be more easily configured in things like Docker containers.

This is really simple, and already supported by the library used, we just need to figure out appropriate names.

https://oclif.io/docs/flags is the documentation for the flags library, the key feature being env.

We'd just need to add that to each of the flags in https://github.com/codercom/code-server/blob/master/packages/server/src/cli.ts#L20 like the following:

host: flags.string({ char: "h", default: "0.0.0.0", env: "HOST" }),

Thoughts are appreciated on this :).

[kylecarbs]

LGTM. Feel free to throw up a PR for this, would be a great addition!

[aslafy-z]

I'd love to see this land as I'm building a Helm Chart for code-server. It would consistently help setting settings and secrets.

[Omeryl]

@aslafy-z I have a PR I'm going to submit tonight, I just had other stuff come up. Have to finish my workday first :).

[amanvishnani]

Any update on this?

[deansheather]

No updates, sorry. Feel free to open a PR!

[zabawaba99]

As of https://github.com/cdr/code-server/commit/1697cc32a335b0510725621be6023cf2ec7cc497, the flags package is no longer used. That being said should still be relatively simple to allow people to use environment variables instead of options. I'll open a PR shortly to expose a few and we can have the discussion there as to what the others should be called.

[nhooyr]

This should be filed against VS Code, we just use their parser.