Hendrik7 / psi-dev

Automatically exported from code.google.com/p/psi-dev
0 stars 0 forks source link

Не показывается пароль из приглашения в конференцию #557

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Psi+ 0.16.204 под Win7 32bit.

Мне посылают приглашение в комнату. Пароль 
в запросе приходит, его видно в XML консоли: <x 
xmlns="jabber:x:conference" password="123" jid="...."/>, однако 
когда я открываю сообщение никакого пароля 
не показывается, и при двойном клике по 
вложению "Приглашение в ..." открывается 
диалог входа в конфу с пустым полем пароля.

Original issue reported on code.google.com by linuxsqu...@gmail.com on 3 Sep 2013 at 7:22

GoogleCodeExporter commented 9 years ago
В тексте приглашения пароль присутствует:
xxxxx@qqq.ru/main/rs04 приглашает вас в комнату 
qwer22@conference.jabber.ru, пароль: '1234'
Или багу уже исправили, или мне просто не 
удалось воспроизвести. Версия - последняя 
на текущий момент.
А вот в диалоге, после двойного клика, 
пароль действительно отсутствует. Он там 
нужен?

Original comment by li...@mail.ru on 17 Jan 2014 at 4:38

GoogleCodeExporter commented 9 years ago
конечно нужен, не вбивать же в ручную, вдруг 
криптоманьяки в свою конференцию инвайтят

Original comment by korobcoff on 17 Jan 2014 at 5:30

GoogleCodeExporter commented 9 years ago
> конечно нужен, не вбивать же в ручную, 
вдруг криптоманьяки в свою конференцию 
инвайтят
копипаст уже не рулит? :)

В принципе, багу я локализовал. Там тупо 
поленились прописать, т.к. передача пароля 
в аргументах была уже предусмотрена. Или 
предусмотрели передачу пароля на форму уже 
позже, но не дописали для приглашения.

Original comment by li...@mail.ru on 17 Jan 2014 at 5:45

GoogleCodeExporter commented 9 years ago
Назвать это багфиксом - себе польстить :)
--- psi.orig/src/eventdlg.cpp
+++ psi/src/eventdlg.cpp
@@ -2032,10 +2032,10 @@ void EventDlg::updateReadNext(PsiIcon *n
        }
 }

-void EventDlg::actionGCJoin(const QString &gc, const QString&)
+void EventDlg::actionGCJoin(const QString &gc, const QString &passw)
 {
        Jid j(gc);
-       d->pa->actionJoin(j.withResource(""));
+       d->pa->actionJoin(j.withResource(""), passw);
 }

 void EventDlg::updatePGP()

Может сразу в апстрим пульнуть?

Original comment by li...@mail.ru on 17 Jan 2014 at 5:52

GoogleCodeExporter commented 9 years ago
Патч принят в апстрим.
https://github.com/psi-im/psi/commit/f7586a6065be01f93f748322d3ea895363405e75

Original comment by li...@mail.ru on 17 Jan 2014 at 8:26