search

stakwork / sphinx-tribes

Backend for sphinx tribes and bounties. The bounty platform pays out in bitcoin. Sign up with Sphinx Chat, complete a bounty, and earn bitcoin! Go to our website for available bounties.
https://community.sphinx.chat/bounties
33 stars 60 forks source link

Merge different env vars for ADMINS to one #1183

Closed kevkevinpal closed 10 months ago

kevkevinpal commented 10 months ago

I noticed that we have two ENV vars ADMINS and ADMIN_PUBKEYS we should only use one of these in our codebase otherwise it might get confusing

ADMIN_PUBKEYS https://github.com/stakwork/sphinx-tribes/blob/master/handlers/auth.go#L18

ADMINS https://github.com/stakwork/sphinx-tribes/blob/master/config/config.go#L33

This seems a bit confusing as they should be the exact same values

elraphty commented 10 months ago

This PR fixes this https://github.com/stakwork/sphinx-tribes/pull/1192, I removed ADMIN_PUBKEYS env, closing.