WEEE-Open / weeelab-telegram-bot

Telegram bot for querying the weeelab log and some additional cool stuff.
GNU General Public License v3.0
8 stars 4 forks source link

Cache users.json somehow #28

Closed lvps closed 4 years ago

lvps commented 5 years ago

Every command checks for permissions.

Every check for permissions checks the user list.

Every check to the user list is a connection to OwnCloud, a download and some JSON parsing.

It's right here: https://github.com/WEEE-Open/weeelab-telegram-bot/blob/09f4f9e3ea4218edf1d3dcd4420cc0ec1354d7a0/weeelab_bot.py#L227-L231

Cannot be done just once since we may add/remove allowed users at any time, but there are better strategies:

lvps commented 4 years ago

Converted to LDAP and done