search

southbridgeio / redmine_2fa

Two-factor (sms/telegram) authentication for Redmine
https://southbridge.io
MIT License
42 stars 17 forks source link

Is redmine_telegram_common actually a requirement? #18

Closed kostajh closed 7 years ago

kostajh commented 7 years ago

Hi! Thanks for this wonderful plugin.

I'm using it only for Google Authenticator functionality. The plugin seems to work just fine without installing redmine_telegram_common. I just wanted to confirm that I'm not missing something. Perhaps the README could be updated to indicate that redmine_telegram_common is a requirement only if you want to use the Telegram auth functionality?

arturtr commented 7 years ago

It is required for OtpBotWebhookController. It needs only for telegram, but rails loads this controller (with other controllers) on rails start.

You can try to remove redmine_telegram_common, but I think that you'll catch Uninitialized constant error after it. But you can try :)