v2 breaks support for configuration lists on environment variables about traefik-forward-auth

[thomseddon]

Thank you for the bug report! I tried to do as much testing as I could in #26 but inevitably some bugs may slip through, the goal of v2 was to be fully backwards compatible so I'll look to push a fix ASAP.

I've just added some extra tests in 72fc88a and everything seems to work as expected.

Please could you provide an example config and let me know the behavior before/after, I'll look into it right away!

[tardfree]

No problem.

The config I was using which I saw the bug on had the following environment variables set (viewed from docker inspect, this is all of them except PATH, names changed too)

"DOMAIN=mydomainname.net,anothername.com,athirdname.com",
"AUTH_HOST=login.mydomainname.net",
"COOKIE_DOMAINS=services.mydomainname.net",
"SECRET=secure string here",
"COOKIE_SECURE=false",
"CLIENT_ID=clientidhere",
"CLIENT_SECRET=secrethere",

If it helps, the domain I was trying to authenticate with was the middle one of the 3.
To work around the issue I've redeployed with a single domain in the list (until I'm back in my home country) and this is working again for me (for that single domain though).

[tardfree]

Oops didn't answer the rest of your questions.
Before behaviour was users in the middle domain were able to authenticate as expected and get redirected to the intended page.
After the update, I received the "Not authorized" string in the browser, and the log message in the first post.
I'm not sure if any of my users in the other domains (first and third) have ever logged in, so that might not have been working.

[thomseddon]

Brilliant, I think I found the bug, I had incorrectly thought the v0 config option was called domains, however I can now see it was called domain!

I have fixed CSV support for domain and added tests for both the flag and the env var. I've also removed the domains config as this was never needed!

[thomseddon]

*I've pushed the fix in 3e92400

Thank you for your bug report and sorry for causing you an issue whilst you were away!

[tardfree]

Thank you for the very quick fix and this simple tool. :)