talk-android icon indicating copy to clipboard operation
talk-android copied to clipboard

switching accounts fails / wrong username shown in account switcher

Open mahibi opened this issue 3 years ago • 11 comments

Steps to reproduce

there exist 2 users: userA and userB

  1. login with userA
  2. go to account switcher
  3. BUG: avatars are shown correctly, but userA has the name of userB
  4. keep userA selected. go to some conversation and send a message
  5. BUG: message is sent as userB
  6. go back to account switcher and select userB
  7. go to the conversation and send a message
  8. BUG: message is sent as userA

this doesn't always happen but sometimes.

Expected behaviour

  • names in account switcher match the avatars
  • actions are not executed as another user

i guess this is caused by the room migration..

Device information

Talk version: master (>= 14.2.0 Alpha 02)

Server information

sermo

mahibi avatar Jul 14 '22 16:07 mahibi

@mahibi are both users on the same server or different ones? I tested with 3 users on 3 servers and don't have an issue there.

AndyScherzinger avatar Jul 14 '22 21:07 AndyScherzinger

both on the same server (sermo)

mahibi avatar Jul 15 '22 06:07 mahibi

just reproduced this with 14.1.0 RC2 ! this time the names in the account switch dialog were correct but the messages were still send as the wrong user!

mahibi avatar Jul 15 '22 11:07 mahibi

i debugged talk app 14.1 RC2 (on sermo, 25.0.0.3) and i see that all information is sent correctly, but somehow the response says it's from an other user. (i send the message "Test22" as user "marcel2", but the response says it's from "Marcel")

grafik

So i suspect it's an error on server side.. any ideas @nickvergessen ?

mahibi avatar Jul 15 '22 13:07 mahibi

@mahibi but it works with 14.0.2 - no?

AndyScherzinger avatar Jul 15 '22 13:07 AndyScherzinger

same error with talk app 14.0.2 on sermo. so either i proved it's a bug present in the app or (which i assume as described) on sermo... now trying with older servers..

mahibi avatar Jul 15 '22 14:07 mahibi

same error with 14.0.0 with personal server on 24.0.2 while it works fine with 13.0.0 so it is not the DB technology switch but something else.

AndyScherzinger avatar Jul 16 '22 11:07 AndyScherzinger

--> issue must be client side

AndyScherzinger avatar Jul 16 '22 11:07 AndyScherzinger

as expected, and now approved: this happens only for accounts that are on the same server. So this should not affect too many users, anyway it's a serious bug.

mahibi avatar Jul 18 '22 10:07 mahibi

Some thinking and digging broad me to testdriving cookie clearing, see #2229 which leads to the correct expected results whenever you clear all cookies before the next server call, seems the server's response resets the cookie state to a "wrong" one, so without clearing all follow-up server calls are wrong again.

AndyScherzinger avatar Jul 20 '22 15:07 AndyScherzinger

@jld3103 was able to reproduce this once! So we tried again but now fail to reprduce it another time...

Talk app version 15.1.0 Alpha 03 tested with 2 accounts on the same localhost

fyi @AndyScherzinger

mahibi avatar Sep 30 '22 13:09 mahibi

never experienced this again. closing..

mahibi avatar Mar 01 '23 14:03 mahibi