isamert / scli

a simple terminal user interface for signal messenger (using signal-cli)
GNU General Public License v3.0
448 stars 39 forks source link

Adapt to signal-cli v0.13.0 #220

Closed nelijuc closed 7 months ago

nelijuc commented 7 months ago

Adapt to signal-cli v0.13.0 due to daemon log refactor (https://github.com/AsamK/signal-cli/commit/7e9940be4ac1d3b7e19bcbd1e92b8db436415195)

h3artbl33d commented 7 months ago

Hi,

Does this work for you? I have applied your patch, but scli still keeps hanging on Initializing signal-cli daemon. I am able to send out messages succesfully, but it seems that the DBus changes somehow impact the fetching in scli.

exquo commented 7 months ago

@nelijuc Thank you!

exquo commented 7 months ago

@h3artbl33d Works for me. Can you try it with logging

scli --debug

and share the output (in ~/.local/share/scli/log by default), removing any personal info (phone nums, etc).

0x09AF commented 7 months ago

Here is mine. Same symptoms

INFO:root:scli ?
DEBUG:root:callf: `['signal-cli', '-u', '+12345678910', '--output=json', 'daemon', '--dbus']`
DEBUG:urwid.event_loop.main_loop.MainLoop:Starting event loop 'SelectEventLoop' to manage display.
DEBUG:urwid.event_loop.select_loop.SelectEventLoop:Waiting for input: timeout=0.0
DEBUG:urwid.event_loop.select_loop.SelectEventLoop:No input in timeout, calling scheduled <bound method MainLoop.entering_idle of <urwid.event_loop.main_loop.MainLoop object at 0x7e20c2e354d0>>
DEBUG:urwid.event_loop.main_loop.MainLoop:Screen size recalculated: (281, 50)
DEBUG:urwid.display._posix_raw_display.Screen.draw_screen:Drawing screen with size (281, 50)
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.display._posix_raw_display.Screen:Undefined attribute: ''
DEBUG:urwid.event_loop.select_loop.SelectEventLoop:Processing input
DEBUG:urwid.event_loop.select_loop.SelectEventLoop:Waiting for input: timeout=0.0
DEBUG:urwid.event_loop.select_loop.SelectEventLoop:No input, entering IDLE
DEBUG:urwid.event_loop.select_loop.SelectEventLoop:Processing input
DEBUG:urwid.event_loop.select_loop.SelectEventLoop:Waiting for input: timeout
DEBUG:urwid.event_loop.select_loop.SelectEventLoop:Processing input
INFO:root:daemon_log: INFO  DaemonCommand - Starting daemon in single-account mode for +12345678910
DEBUG:urwid.event_loop.select_loop.SelectEventLoop:Waiting for input: timeout=0.0
DEBUG:urwid.event_loop.select_loop.SelectEventLoop:No input, entering IDLE
DEBUG:urwid.event_loop.select_loop.SelectEventLoop:Processing input
DEBUG:urwid.event_loop.select_loop.SelectEventLoop:Waiting for input: timeout
exquo commented 7 months ago

@0x09AF Thanks for reporting! This looks like a separate problem, caused by a recent version of urwid (>=2.6.0). I've opened a new issue to track this. In the meantime, you can revert to an earlier urwid (<=2.5.3).

h3artbl33d commented 7 months ago

@exquo

Thanks! Downgrading the urwid version has fixed my issue as well (was about to do a debug, tried this first).

exquo commented 7 months ago

Scli's master branch should now work with the new versions of urwid. So it should not be necessary to downgrade.