mod_scam_upgrade causing SQL transaction abort

[mightyBroccoli]

Environment

• ejabberd version: 24.12
• Erlang version: Erlang (SMP,ASYNC_THREADS) (BEAM) emulator version 13.1.5
• OS: Debian 12.8
• Installed from: source

Configuration (only if needed): grep -Ev '^$|^\s*#' ejabberd.yml

modules:
  mod_scram_upgrade:
      offered_upgrades:
      - sha256
      - sha512

Errors from error.log/crash.log

2024-12-20 17:03:03.436201+01:00 [error] <0.382.0>@ejabberd_sql:check_error/2:1414 SQL query 'Q84048542' at {ejabberd_auth_sql,{173,9}} failed: <<"Incorrect string value: '\\xA4=\\xB5\\x97|(...' for column `ejabberd_magicbroccoli`.`users`.`password` at row 1">>
2024-12-20 17:03:03.436449+01:00 [error] <0.382.0>@ejabberd_sql:outer_transaction/3:612 SQL transaction restarts exceeded
** Restarts: 10
** Last abort reason: <<"Incorrect string value: '\\xA4=\\xB5\\x97|(...' for column `ejabberd_magicbroccoli`.`users`.`password` at row 1">>
** Stacktrace: [{ejabberd_sql,restart,1,
                              [{file,"/usr/local/src/ejabberd/src/ejabberd_sql.erl"},
                               {line,223}]},
                {ejabberd_sql,outer_transaction,3,
                              [{file,"/usr/local/src/ejabberd/src/ejabberd_sql.erl"},
                               {line,599}]},
                {ejabberd_sql,run_sql_cmd,4,
                              [{file,"/usr/local/src/ejabberd/src/ejabberd_sql.erl"},
                               {line,536}]},
                {p1_fsm,handle_msg,10,
                        [{file,"/usr/local/src/ejabberd/_build/default/lib/p1_utils/src/p1_fsm.erl"},
                         {line,582}]},
                {proc_lib,init_p_do_apply,3,
                          [{file,"proc_lib.erl"},{line,240}]}]
** When State == {state,<0.392.0>,mysql,
                        {10011006,'MariaDB',0},
                        0,<<"magicbroccoli.de">>,
                        {{[],[]},0,50000},
                        undefined,60000}

Bug description

After activating the module and connecting with Monal this error is produced. The "error" is fixed by disableling the module.

CC @weiss

[licaon-kter]

Isn't this still #4301 ?

[mdosch]

@licaon-kter Maybe rename the issue to reflect what it is about? When searching issues it looks like 4301 this is just about documentation/example config.

[licaon-kter]

Done, got a better text? :)