Gajim - 2024-04-12


  1. paper67

    Hello

  2. avva3814

    is Gajim the same as jabber

  3. vanis

    avva3814: Gajim is a xmppp client

  4. bot

    lovetox pushed 1 commits to branch gajim/master refactor: Rewrite archive database code - https://dev.gajim.org/gajim/gajim/-/commit/fc147adbe0713a28f537ae98556267a59b357b7b

  5. cal0pteryx

    It's happening!!1!

  6. rom1dep

    πŸŽ‰

  7. avva3814

    a

  8. lovetox

    yeah good luck with the migration !

  9. lovetox

    we create a backup of the old db before migration, so it should not be hard to go back *if* it fails for some reason

  10. cal0pteryx

    and please post the report it shows (if it crashes)

  11. bot

    lovetox pushed 1 commits to branch gajim/master cfix: Correctly determine nickname in private messages - https://dev.gajim.org/gajim/gajim/-/commit/e8fef3d72d47c667058735f888365a7e720d5b75

  12. rom1dep

    my logs are, like 1.8G

  13. rom1dep

    how is the migration story? Should I just checkout, pip install -e . --user and ./launch.py ?

  14. lovetox

    yes, but this will take like half an hour

  15. lovetox

    do you normally call pip install?

  16. rom1dep

    it crashes before attempting anything apparently > sqlite3.OperationalError: no such table: remote

  17. lovetox

    can you get me a full log?

  18. lovetox

    did it do the backup ?

  19. rom1dep

    yep

  20. rom1dep

    hold on

  21. rom1dep

    Pastebin’s SMART filters have detected potentially offensive or questionable content in your Paste.

  22. rom1dep

    SMART uppercase it must be, then

  23. rom1dep

    http://tamytro.org/tmp/gajim_new_db_model.txt

  24. lovetox

    so did it create a backup .bak file?

  25. rom1dep

    not even

  26. lovetox

    in .local/share/gajim

  27. lovetox

    its weird its like the migration is skipped, can you check the user_version field of your database?

  28. rom1dep

    in logs.db?

  29. lovetox

    yes

  30. rom1dep

    I see only 4 tables in there

  31. lovetox

    yes but the database has a user_version field

  32. lovetox

    its not in a table

  33. rom1dep

    I didn't know that was a thing

  34. lovetox

    do you use a GUI app for looking at the db?

  35. cal0pteryx

    Sqlite browser shows that field for example

  36. lovetox

    or simply execute

  37. lovetox

    PRAGMA user_version

  38. rom1dep

    says 8

  39. lovetox

    Oo

  40. lovetox

    Set it back to 7

  41. lovetox

    PRAGMA user_version = 7

  42. lovetox

    and then try again, but gather logs

  43. cal0pteryx

    rom1dep: did you run the storage branch at some point?

  44. rom1dep

    I may have accidentally; I have a script that updates gajim and its dependencies but that sometimes breaks when branches diverge (stuff gets force pushed in weird ways)

  45. rom1dep

    ok, now I have a GUI telling me that it failed instead πŸ™‚

  46. rom1dep

    I've got to go take a phone call, I'll be back later

  47. rom1dep

    It crashed on: File "/home/rom1dep/Binaires/Network/gajim/gajim/common/storage/archive/migration.py", line 258, in _process_message_row account_jid = JID.from_string(log_row.account.jid) ^^^^^^^^^^^^^^^^^^^ AttributeError: 'NoneType' object has no attribute 'jid'

  48. lovetox

    mhm odd, messges in the database that link to no account

  49. bot

    lovetox pushed 1 commits to branch gajim/master cfix: Migration: Don’t fail on missing jids - https://dev.gajim.org/gajim/gajim/-/commit/690ed499681d21985e0c05fc396c471f5b6da52b

  50. lovetox

    rom1dep, i pushed a mitigation of that problem

  51. lovetox

    please pull and try again

  52. rom1dep

    migration going on πŸ™‚

  53. rom1dep

    10%

  54. rom1dep

    25%

  55. lovetox

    after migration, quit gajim normally, this can take also a long time, because it will free up space from your database

  56. rom1dep

    50%

  57. rom1dep

    I have a buffer full of warnings for message not being migrated due to missing jid or account, I'm curious to see how many messages will come up missing πŸ™‚

  58. cal0pteryx

    My db was smaller after migrating. I'm curious about how much space is occupied by the migrated db, rom1dep

  59. cal0pteryx

    (I migrated before that fix)

  60. rom1dep

    let me vaccuum the backup for fairness

  61. rom1dep

    80%

  62. rom1dep

    vacuuming didn't reduce size noticeably

  63. rom1dep

    100%

  64. rom1dep

    https://upload.tamytro.org/upload/a3849b367f4dd4cedd5ad6f64afa7645b7d4f834/M9CTfOjdyN92MMotBeMyNrzbzh6xirDRb7ChpHvB/3e472291-b652-4e0f-9c63-50067ec9a54e.png

  65. lovetox

    .. ah damn now the same shit in that table

  66. bot

    lovetox pushed 1 commits to branch gajim/master cfix: Migration: Don’t fail on missing jids in mam migration - https://dev.gajim.org/gajim/gajim/-/commit/a26ce1180d14d9f5dd98c7bbe100fbeb85fa1d8a

  67. lovetox

    rom1dep, so now try again please

  68. rom1dep

    here we go again πŸ™‚

  69. bot

    lovetox pushed 1 commits to branch gajim/master cfix: Message: Store resource for private messages - https://dev.gajim.org/gajim/gajim/-/commit/f4336f45edfb06d826ab1902cd2f25f5838c8f44

  70. rom1dep

    done, and new crash

  71. rom1dep

    https://upload.tamytro.org/upload/a3849b367f4dd4cedd5ad6f64afa7645b7d4f834/xp8zrqxL88hFxFO116h4oPN7aelDv50yf1rn0yH3/6ae9a782-a331-43be-8973-c065c6d1e95d.png

  72. cal0pteryx

    rom1dep, thanks for testing, this is really useful

  73. rom1dep

    it isn't much, thanks to you guys for the great work πŸ™‚

  74. lovetox

    that comes of as sarcastic if you say this after the third failed attempt :D

  75. rom1dep

    hahaha, common, I know what I'm getting into by running from master :)

  76. rom1dep

    and it's not like I paid a lot of money or anything ^^

  77. lovetox

    hm rom1dep can you export me the table last_archive_message

  78. lovetox

    hm rom1dep can you export me the table "last_archive_message"

  79. rom1dep

    sent a url in PM

  80. lovetox

    thanks

  81. rom1dep

    I'm trying to fixup my logs in the interim, the last log line I have with a missing account_id dates back to late 2017

  82. rom1dep

    (also, I'm keeping the "broken" original db for those migration tests, of course)

  83. lovetox

    but we would not migrate these anyway

  84. lovetox

    so not need to fix it

  85. lovetox

    or do you want to remap it

  86. rom1dep

    I want to remap it

  87. rom1dep

    what does the account_id value maps to?

  88. rom1dep

    I have, like 7 distinct values for it, from 1 to 221

  89. bot

    lovetox pushed 1 commits to branch gajim/master cfix: Migration: Make mam state migration more resilient - https://dev.gajim.org/gajim/gajim/-/commit/f1dbec4ff5bb89f8524584841d9827030c180729

  90. lovetox

    to the jids table

  91. rom1dep

    should I attempt another migration ? πŸ™‚

  92. lovetox

    it should contain the id in the jids table that is your account jid

  93. lovetox

    yeah i hope your error is now fixed

  94. rom1dep

    https://upload.tamytro.org/upload/a3849b367f4dd4cedd5ad6f64afa7645b7d4f834/oF6cwBSKXDi5y2wXLygWSVkAaxBnxljobewdqfgI/d4e49358-b914-45b1-a468-9f52a4b3024c.png

  95. rom1dep

    look at that πŸ™‚

  96. cal0pteryx

    great! how long did it take for 1.8gb?

  97. rom1dep

    new size is 507M down from 1.8G

  98. rom1dep

    about 10min

  99. rom1dep

    https://upload.tamytro.org/upload/a3849b367f4dd4cedd5ad6f64afa7645b7d4f834/duYv6S4TEvP5ivYIt8wXMNW3vMfsuiQrGZOrU9MO/bbe7a8a7-6bb3-46e4-bccf-3125ad572a34.png

  100. rom1dep

    apparently there might be some dedup issues

  101. ngn

    hello, I am trying to setup PGP encryption with Gajim, I already have a PGP key and I want to use it, how can I import this key? Also which plugin should I use? PGP or OpenPGP? What is the difference?

  102. lovetox

    only PGP allows you to use a key

  103. lovetox

    check the plugin config dialog

  104. lovetox

    in the plugin config i think there is a option to define a key

  105. lovetox

    rom1dep, weird how that is the only message dupliated

  106. lovetox

    you can delete that one message over the menu

  107. lovetox

    maybe the mam archive state migration is still not perfect somehow

  108. ngn

    lovetox, thanks! and one last thing, how do i assign keys to other contacts?

  109. lovetox

    if you select the encryption iside the chat

  110. lovetox

    there comes a shield icon

  111. lovetox

    if you click it i think that shows a dialog where you can assign a key

  112. ngn

    > if you select the encryption iside the chat > there comes a shield icon oh I see it, thank you so much!

  113. fjklp

    is the deduplication considering the same message from different accounts as duplicates?

  114. lovetox

    there is no deduplication

  115. fjklp

    That works too. I'm just lost in the conversation, carry on

  116. fjklp

    the db migration makes me feel uneasy πŸ™‚

  117. rom1dep

    I fixed my logs DB and it's still much smaller after migration :)

  118. rom1dep

    fjklp last messages still appears duplicated

  119. rom1dep

    fjklp last messages appears duplicated

  120. rom1dep

    but my message from earlier does not

  121. rom1dep

    could it have to do with messages being received while MAM is fetching?

  122. lovetox

    is it reproduceable? otherwise i would say its some kind of hicup with migrating the mam state

  123. rom1dep

    hard to tell as such

  124. rom1dep

    OTOH, I'm missing a whole bunch of biboumi messages right now

  125. rom1dep

    (W) gajim.c.m.message (tamytro.org) Received message without message id

  126. rom1dep

    are messages without message id dropped or not displayed in the new implementation, somehow?

  127. rom1dep

    I can almost read the full backlog from my terminal, other clients show it, but not gajim ;Γ 

  128. rom1dep

    I can almost read the full backlog from my terminal, other clients show it, but not gajim :)

  129. lovetox

    i think its the conversation message receipt

  130. lovetox

    not a real message

  131. lovetox

    rom1dep, no reason comes to mind why a biboumi message would not be migrated

  132. rom1dep

    this isn't about migration

  133. rom1dep

    those are new messages

  134. rom1dep

    MAM fetched

  135. lovetox

    but if you have the jid you could look in the old db how the messages look that are not migrated

  136. lovetox

    ah

  137. rom1dep

    i.e. new to gajim

  138. lovetox

    you were joined this chat when the last time?

  139. rom1dep

    yesterday

  140. lovetox

    1 day should be requested in any case

  141. lovetox

    hmm

  142. rom1dep

    and as I said, the messages are requested, and printed in the logs

  143. rom1dep

    not in the messagewindow

  144. lovetox

    and the log says not more than the message id stuff?

  145. rom1dep

    2024-04-12T23:06:15 (W) gajim.c.m.message (account) Received message without message id 2024-04-12T23:06:15 (W) gajim.c.m.message (account) <message xmlns="jabber:client" from="#room%irc.ircserv.org@blah/nick" type="groupchat" to="me"><body>X said Y</body></message>

  146. rom1dep

    every now and then I get a "Received message without text"

  147. rom1dep

    but seems unrelated

  148. lovetox

    yeah this is then really a problem

  149. lovetox

    biboumi is a pain

  150. rom1dep

    I thought they came to the point of supporting a decently recent/featured MAM:version, but anyhow, this feels like a new behavior because messages from biboumi were properly displayed and archived before

  151. lovetox

    but i just checked and the biboumi instance on jabber.fr has message ids on the messages

  152. lovetox

    can it be that this particular instance you are using is ancient?

  153. rom1dep

    it's the one I run myself, it's the most recent or second most recent commit from the repo

  154. lovetox

    ah mom i see

  155. lovetox

    but its not messages in a chat

  156. rom1dep

    can be that jabberfr has patches on top of it, though

  157. lovetox

    i see this on the irc network messages

  158. lovetox

    like the irc server sends you stuff that you should register a nick and something like that, and this has no message ids

  159. lovetox

    if you go to view -> XML Console -> Log

  160. lovetox

    thats a new tab you see errors there

  161. lovetox

    and there is the full stanza that has no message id

  162. lovetox

    can you send me one

  163. rom1dep

    I can see the logs, how do I relate that to the stanza? Or you mean, I wait for a new log line to appear and catch the stanza at that time?

  164. rom1dep

    anyhow, I'm much less worried about those than I am about no biboumi message showing up anymore

  165. rom1dep

    would I be able to see from the logs.db whether those used to have message IDs ?

  166. lovetox

    with biboumi messages you mean messages in a channel, or from the irc server?

  167. rom1dep

    in a channel

  168. lovetox

    the stanza is printed under the error message

  169. lovetox

    in the Log tab

  170. rom1dep

    new messages seem to appear (moreover, with ID), but MAM messages do not

  171. lovetox

    but this can have different reasons

  172. lovetox

    i doubt biboumi does not add a message id

  173. lovetox

    can i join this irc channel you are talking about?

  174. rom1dep

    can I trash the room's history and re-fetch it on demand ?

  175. rom1dep

    well, in fact, I have the same problem on 2 different chanels on 2 different biboumi instances (so, 4 repros)

  176. lovetox

    im investigating

  177. rom1dep

    if I remove a room's history, then leave it, then join it again, would gajim try to fetch some history from MAM?

  178. lovetox

    no

  179. lovetox

    you need delete the row in mam archive state table

  180. rom1dep

    let me try that, then

  181. rom1dep

    so, I can totally repro that, then

  182. rom1dep

    I left the room, cleaned history, cleaned the mam_archive_state, re-joined

  183. rom1dep

    I can see the errors in the logs about "received without message id" and nothing printed in the gajim's messagewindow

  184. bot

    lovetox pushed 2 commits to branch gajim/master cfix: ChatBanner: Don’t fail if there is no resource set - https://dev.gajim.org/gajim/gajim/-/commit/6fc27014cb1fef51136ff22bd524344429f7cab2 cfix: Message: Accept messages without id - https://dev.gajim.org/gajim/gajim/-/commit/3b5f46b95d41063885800187212ca35b6dc026ef

  185. rom1dep

    Calling it a day, thanks for all the bugfixes, gajim seem in a much better shape now πŸ™‚πŸ‘