mirror of
https://github.com/nextcloud/twofactor_gateway.git
synced 2026-04-25 09:05:55 +03:00
[GH-ISSUE #86] Handle Telegram edge cases #25
Labels
No labels
0. to triage
1. to develop
3. to review
blocked
bug
discussion
duplicate
enhancement
enhancement
gateway:signal
gateway:signal
gateway:signal
gateway:sms
gateway:telegram
hacktoberfest
help wanted
invalid
needs info
php
pull-request
question
technical debt
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/twofactor_gateway-nextcloud#25
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @ChristophWurst on GitHub (Aug 7, 2018).
Original GitHub issue: https://github.com/nextcloud/twofactor_gateway/issues/86
In order to stabilize the Telegram integration, it needs a few fixes in its gateway implementation that handle edge cases.
@ChristophWurst commented on GitHub (Aug 28, 2018):
Btw, @djpremier , see the new admin documentation for more info on how to set this up: https://github.com/nextcloud/twofactor_gateway/tree/master/doc/admin#telegram.
@ChristophWurst commented on GitHub (Aug 28, 2018):
Basically
github.com/nextcloud/twofactor_gateway@ce47ec9b67/lib/Service/Gateway/Telegram/Gateway.php (L90).@djpremier commented on GitHub (Aug 28, 2018):
Thanks Christoph, I already started the conversation with the button, in the case of '/start', and still I get the error.
@djpremier commented on GitHub (Aug 28, 2018):
Now I'm on the computer, I can give you more details.
Following @ChristophWurst , the process was basically this:
I installed the app
I have read the documentation
I created the bot and started the conversation '/ start'
I configured via 'occ' the gateway
I went back to my settings panel and added my number
I get the error in the same place and also in the log, as you can see in the screenshots
@ChristophWurst commented on GitHub (Aug 28, 2018):
Did you enter your phone number or the chat id from the id bot?
@djpremier commented on GitHub (Aug 28, 2018):
Phone number
@ChristophWurst commented on GitHub (Aug 28, 2018):
Get your ID via this bot and enter that is identifier.
@djpremier commented on GitHub (Aug 28, 2018):
Thank you, it worked! But it is strange that you have placed the requested message to enter the phone number, since what should be placed is chat_id.
Another alternative is to send the message of / start and access:
https://api.telegram.org/bot<token>/getUpdatesThere you will have access to id
I think it would be interesting if possible to mount this mini tutorial there in docs 😉
@ChristophWurst commented on GitHub (Aug 28, 2018):
That's exactly what this app does internally. But with that info alone, you have no mapping of Nextcloud and Telegram users. Users have to enter their chat id.
The steps will be documented in the UI starting with the next release. Ref #82.