Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add DISCORD_APPID to docs + Better error message for missing env variables #131

Closed
UnlimitedCookies opened this issue Feb 3, 2022 · 5 comments · Fixed by #132
Closed
Assignees
Labels
❓ help wanted Extra attention is needed

Comments

@UnlimitedCookies
Copy link

Description

The bot doesn't start and shows this message instead and immediately exits:

Error: NotPresent

I've tried both the ghcr.io image and also compiled it from git with the newest rust nightly version.
The DISCORD_TOKEN env variable was set.

Expected vs. Observed

It's expected to start or show a helpful error message.

Repro Steps

  1. Download the current docker image and run it
  2. Start the bot

Environment

Key Value
Operating System Arch Linux
@afonsojramos
Copy link
Collaborator

Hi @UnlimitedCookies!

The Error: NotPresent shows up when one of the environment variables is not set.
Did you also set DISCORD_APPID? You need both env vars.

@aquelemiguel aquelemiguel added 🐛 bug Something isn't working ❓ help wanted Extra attention is needed labels Feb 3, 2022
@afonsojramos afonsojramos removed the 🐛 bug Something isn't working label Feb 3, 2022
@UnlimitedCookies
Copy link
Author

Oh, yep - that fixed the issue.
You should definitely add this to the documentation. Also, the error message could be a little more helpful imo.

@UnlimitedCookies UnlimitedCookies changed the title Bot not starting: Error: NotPresent Add DISCORD_APPID to docs + Better error message for missing env variables Feb 3, 2022
@afonsojramos afonsojramos self-assigned this Feb 4, 2022
@afonsojramos
Copy link
Collaborator

Truly thought we had that, will update!

Thanks for the issue @UnlimitedCookies!

@UnlimitedCookies
Copy link
Author

Awesome! Perhaps you could also add it to the Readme.md for now?

Later down the road, I suggest adding a wiki page with all the possible environment variables.

@afonsojramos
Copy link
Collaborator

I already updated the Wiki (Step 7), but will also make a revision to the readme.md in my PR, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
❓ help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants