Am 14/12/2023 um 10:55 schrieb Dominik Csapak: > logging in with totp on mobile was broken with these two commits: > > pve-manager: > 509d7a20 ("mobile ui: implement dummy message box and scrip loader") > and > pve-access-control: > cb64967 ("api: drop old verify_tfa api call") > > the pve-manager one overwrote the Ext.MessageBox and Ext.Msg classes and > thus removed the Ext.MessageBox.OKCANCEL constant that represented the > buttons of popup messages (without those no buttons on message boxes > where shown). > > This override did not work as intended, as we still showed the message > box by accident, because at that point the Ext.MessageBox was already > initialized (so it was overwritten), but Ext.Msg was not (this happens > later). > > and the pve-access-control removed the old tfa verify api (which is now > done via the /access/ticket api) > > so to fix that, we have to adapt to the api changes and restore the > stock Ext.MessageBox and Ext.Msg classes by removing the overrides > (i couldn't find where we would need those) > > we still cannot handle u2f or recovery methods though > > Signed-off-by: Dominik Csapak <d.csa...@proxmox.com> > --- > www/mobile/Login.js | 8 ++++++-- > www/mobile/Toolkit.js | 5 ----- > 2 files changed, 6 insertions(+), 7 deletions(-) > >
applied both patches, thanks! _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel