processone / ejabberd-contrib

Growing and curated ejabberd contributions repository - PR or ask to join !
http://ejabberd.im
248 stars 137 forks source link

mod_irc crash #324

Open Void-Seeker opened 9 months ago

Void-Seeker commented 9 months ago

mod_irc crashes on launch, this is in the console on ejabberdctl live.

2023-09-15 08:08:57.873103+03:00 [critical] ejabberd initialization was aborted because a module start failed.
2023-09-15 08:08:57.872634+03:00 [error] CRASH REPORT:
  crasher:
    initial call: mod_irc:init/1
    pid: <0.512.0>
    registered_name: []
    exception error: undefined function gen_mod:get_opt_hosts/2
      in function  mod_irc:init/1 (mod_irc.erl, line 98)
      in call from gen_server:init_it/2 (gen_server.erl, line 851)
      in call from gen_server:init_it/6 (gen_server.erl, line 814)
    ancestors: [ejabberd_gen_mod_sup,ejabberd_sup,<0.128.0>]
    message_queue_len: 0
    messages: []
    links: [<0.424.0>]
    dictionary: []
    trap_exit: true
    status: running
    heap_size: 2586
    stack_size: 28
    reductions: 258
  neighbours:

Perhaps, ejabberd api has changed? I think that function only takes one argument in current ejabberd version.

licaon-kter commented 9 months ago

Please use Biboumi instead, this module has been depreciated.

Void-Seeker commented 9 months ago

Biboumi only supports utf8 and iso_8859-1 charsets for irc channel for now. Or at least it ignored charset settings through ad-hoc. Still got eldritch language instead of cyrillic.

licaon-kter commented 9 months ago

This issue? https://lab.louiz.org/louiz/biboumi/-/issues/2411

Void-Seeker commented 9 months ago

This issue? https://lab.louiz.org/louiz/biboumi/-/issues/2411

Yeah, probably... I only connect to one server that only has one room, so setting the encoding for entire transport would be enough for my use case. Unless misunderstood something, but ad-hoc commands for setting encoding per server are reported, but they don't work. And docs for biboumi only mention utf8 and iso_8859-1. So i assume it's still not fully implemented.