Cannot specify multiple redirect domains through ENVIRONMENT_VARIABLES value

[teekennedy]

Hi,

Thanks for creating this project. It's a nice ingress wrapper that has been working well for me 😄

I noticed this line in the logs of my anubis pod:

REDIRECT_DOMAINS is not set, Anubis will only redirect to the same domain a request is coming from, see https://anubis.techaro.lol/docs/admin/configuration/redirect-domains

I went to set REDIRECT_DOMAINS to my root domain as well as any subdomain of that root domain, e.g. REDIRECT_DOMAINS:example.com,*.example.com. Unfortunately, I can't set a value like this through ENVIRONMENT_VARIABLES since the comma is reserved as a separator between different environment variables.

I tried it anyway just to see what would happen, and as I expected it errored out:

failed to run: env: parse error on field "EnvironmentVariables" of type "map[string]string": "*.example.com" should be in "key:value" format

As a workaround, I'm going to add REDIRECT_DOMAINS to a secret and use ENV_FROM_SEC to pass the value to anubis. I made this issue mainly so you are aware that the conflict exists.

Perhaps you could have the ingress controller pass environment variables to anubis by prefix? For example, setting ANUBIS_FOO: bar to the controller results in the controller passing FOO: bar to the anubis pod.

[jaredallard]

Ah, good catch, I'll look at this next week or so during some extended PTO I have. I've been meaning to clean up the env var passing, it's not my favorite. Glad you're unblocked for now at least!

EDIT: Turns out I did not have free time, but I am still planning on this soon!