Environment config not working

[Wishdream]

I'm currently using the docker-compose given via portainer
and provides the following error:

Couldn't read config
Missing field: 'DISCORD_TOKEN'

All my environment vars are set and just following the provided docker-compose file:

version: '3.4'

services:
  aoede:
    image: codetheweb/aoede
    restart: always
    volumes:
      - ./aoede:/data
    environment:
      - DISCORD_TOKEN=discord.token.here
      - SPOTIFY_USERNAME=spotitest
      - SPOTIFY_PASSWORD=notarealpassword
      - DISCORD_USER_ID=targeteduserIDhere

I think it's trying to read the config.toml where it shouldn't exist or somehow got bundled with the docker image

[nodgear]

This issue seems to be related to the fix attempt by on #15
Partially reported on #16

[oleggtro]

Yeah, I think the problem got introduced in #18.
Serdes rename macro somehow messes with figments ability to deserialize the config properly.
I was able to reproduce this issue and fix it by removing the macro.

The easiest way to fix this would probably be ditching figment all together and moving towards dotenv (and relying on a .env file or env vars for configuration). What do you think @codetheweb?

[oleggtro]

The issues are not related btw :)

[oleggtro]

#25 should fix your issue @Wishdream.
Please wait until a build for 87dc75e is published (or build it yourself if you're able to) and try the fix

---

The easiest way to fix this would probably be ditching figment all together and moving towards dotenv (and relying on a .env file or env vars for configuration). What do you think @codetheweb?

I don't know if thats really required. I don't really like the fix(#25) though.

[Wishdream]

#25 should fix your issue @Wishdream. Please wait until a build for 87dc75e is published (or build it yourself if you're able to) and try the fix

Going to build it myself for now, at least until there's an agreed-upon fix.
Not exactly resolved yet so I'll keep this open for now.

[oleggtro]

Well, I can't merge into the main repo, thats why the fix isn't official yet.
Can you confirm it worked for you?

[codetheweb]

I would rather use .env over .toml but no strong preference.