search

TelepathyIM / telepathy-morse

Telegram connection manager for the Telepathy framework
GNU General Public License v2.0
62 stars 10 forks source link

Unable to use new account #32

Closed ItachiSan closed 7 years ago

ItachiSan commented 7 years ago

As in title: with latest version of telepathy-morse, the initial login (the one with the verification code) doesn't work. Previously, Empathy showed a dialog asking for a password (the initial PIN code), after which worked properly. Right now, this doesn't happen.

Expected

ArchLinux, with latest Empathy and telepathy-morse package. telepathy-morse is built from latest sources, using this PKGBUILD: https://aur.archlinux.org/packages/telepathy-morse-git/

Kaffeine commented 7 years ago

Thanks, I will check it soon. I never checked it with Empathy. If the Debug interface works for you, then the debug log would be very appriciated. Does Empathy show Debug info right on the connection manager started or it waits for connection to be established (which is the case for current version of KDE Telepathy Debugger)?

Kaffeine commented 7 years ago

OK, I have the same issue. I think it is after https://github.com/Kaffeine/telegram-qt/commit/1967d6a37668bc02c6651cdb207c60c4a67e02a0 or https://github.com/Kaffeine/telegram-qt/commit/5a10fcb6786636aaf13f7587ce807dfca5323c85. I tested new authorization with telegram-qt/testApp after the commits, but may be Morse works differently.

Kaffeine commented 7 years ago

I found why my test succeed.

Time ago I introduced built-in list of servers and it seems that I made a typo in the first server, because 1) the server doesn't answer, 2) morse has a similar address. All together: because of the typo in telegram server list, I got connection to the appropriate DC and had no "PHONE_MIGRATE_2" error, which I now have with Morse, which successfully connects to the first server.

So the fix has three steps: 1) Update builtInDc list in TelegramQt. 2) Remove DC override from Morse (actually we need to add an account option to set the DC ip) 3) Fixup phone code requests migration.

Kaffeine commented 7 years ago

Fixed by https://github.com/Kaffeine/telegram-qt/commit/bdaf7bd6eac0d08b3c65347efef42761c6c5bc86, the bad commit is https://github.com/Kaffeine/telegram-qt/commit/1967d6a37668bc02c6651cdb207c60c4a67e02a0

@ItachiSan: You need to update only TelegramQt to get this fixed. Sadly I think that I found yet another issue. Avatars are (mostly) missing for me. Seems to be a regression in TelegramQt again. I'm working hard to release this, so I reworked some parts to be future-proof and compatible with further updates. The reworks leads to regressions. Sorry.