Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jingle-ft doesn't fallback to ibb (or close session) when s5b transfer has failed #784

Open
SkyMaverick opened this issue Sep 28, 2020 · 12 comments

Comments

@SkyMaverick
Copy link

С недавних пор в Conversations есть передача файлов по XEP-065. Пытаюсь использовать связку Converstions + PSI+ linux-build. При передаче (Conversations -> Psi+) Converstions показывает передачу 100% и останавливается на этом, PSI+ показывает размер и имя файла и передачу 0% . В обратную сторону (PSI+ -> Converstions) - не передаёт.

Gajim передача работает в обе стороны, поэтому склоняюсь к мнению, что проблема всё-таки в Psi+. Сервер Prosody, если это важно. Psi+ за NAT.

@Ri0n
Copy link
Member

Ri0n commented Sep 28, 2020

версия Пси+ какая?

@Ri0n
Copy link
Member

Ri0n commented Sep 28, 2020

проверил.
с пси+ на конверс работает нормально.
обратно тоже передает, но пси не поддерживает шифрование на для файл трансфера, так что получает только гарбаж

@Ri0n
Copy link
Member

Ri0n commented Sep 29, 2020

отключил шифрование в Conversation, передал файл. всё пришло без проблем.

короче эта задача только о поддержки шифрования файлов в пси

@Ri0n
Copy link
Member

Ri0n commented Sep 29, 2020

хотя не вру. это баг Conversations. так как тот не сделал дискавери поддерживаемых в пси фичей.

@Ri0n Ri0n closed this as completed Sep 29, 2020
@SkyMaverick
Copy link
Author

Версия Psi+ v1.4.1473 из ppa:psi-plus/ppa . Вот скрин передачи Psi+ -> Conversation (видит, что началась передача, но передача 0%) + лог XML-консоль. И скрин передачи в обратную сторону (Conversations пишет 100%, но если отменить на Psi+ видит, что передача отменена) + лог XML-консоль.

Возможно, проблема может быть в том, что jabber.mkcsrv - локальный домен. С другой стороны тотже gajim это как-то отрабатывает.

Прочитал сообщения. Ок. Буду знать в чём проблема. Спасибо за уделённое время.

@Ri0n
Copy link
Member

Ri0n commented Sep 29, 2020

о вижу из вашей xml консоли что оба послали candidate-error но закрытия сессии не последовало.
короче соединиться не смогли. но сессия должна была обортнуться, сказать что всё пропало и тп ну и вообще с мигрировать на ibb. то что этого не произошло - баг.

а да. я вчера добавил минимальную поддержку received/checksum нотификаций в файлтрансфер. по идее ни на что не должно повлиять, но ошибки в консерве если и были то не должны больше показываться.

короче я багу переименую тогда

@Ri0n Ri0n reopened this Sep 29, 2020
@Ri0n Ri0n changed the title Передача файлов между Conversations и Psi+ по XEP-065 jingle-ft doesn't fallback to ibb (or close session) when s5b transfer has failed Sep 29, 2020
@Ri0n
Copy link
Member

Ri0n commented Sep 29, 2020

и ещё я накидал драфт для менеджера шифрования в пси. пока у нас в чатике. потом смастерю нормальную доку

@Ri0n
Copy link
Member

Ri0n commented Sep 29, 2020

кстати насчет того что не получилось соединиться по s5b может быть багом в конверсейшене. видимо в гаджиме тот же баг.
Я уже указывал разработчику консервы на этот баг и тот согласился что это баг, но не знаю починил он или нет.
если в кратце то консерва неправильно генерирует ключики авторизации для socks5 если соединение идет через прокси.
с разработчиками гаджима я по этому поводу не общался.

@SkyMaverick
Copy link
Author

но не знаю починил он или нет

Версия Conversations 2.9.0 F-Droid. Насколько я понимаю, релизу 5 дней. Так что, видимо, не починил.

@Ri0n
Copy link
Member

Ri0n commented Sep 29, 2020

а между гаджимом и консервой точно по s5b идет а не ibb?

@SkyMaverick
Copy link
Author

Сейчас более подробно проверил с gajim. Получается так:

Если Converstions и gajim находятся в разных подсетях, но без NAT (VLAN в моем случае), то передача отрабатывает нормально (direct connection). Сейчас, имея возможность переключить сеть, проверил Psi+ тоже при отсутствии NAT работает в обе стороны. Стоит оказаться за NAT - все не отрабатывают. Т.е., если я правильно понимаю, mediated сonnection (XEP-0065 пункт 6) не работает.

Проблема касается только файлообмена, т.к. сообщения и станзы ходят спокойно хоть с NAT, хоть без. И, видимо, если я правильно понимаю, Conversations сломавшись с s2b пытается передать по ibb, но psi+ переключиться на ibb не может и Вы правильно диагностировали проблему - psi+ не переходит в fallback-режим.

@Neustradamus
Copy link

@SkyMaverick: Have you tested a recent Psi+ build?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants