tinode / ios

Tinodios: Tinode Messaging Client for iOS
Apache License 2.0
243 stars 107 forks source link

iOS App Crashed when signing up #205

Closed damoncoo closed 1 year ago

damoncoo commented 1 year ago

If you are not reporting a bug, please post to https://groups.google.com/d/forum/tinode instead.

Describe the bug App crashes when signing up

To Reproduce Steps to reproduce the behavior:

  1. Go to 'SignUp'
  2. Click on 'SignUp'
  3. Crashes

Expected behavior No Crash

Screenshots If applicable, add screenshots to help explain your problem.

截屏2023-06-05 23 19 38

Tinodios (please complete the following information):

Server

Additional context No Token

aforge commented 1 year ago

Thanks for reporting. https://github.com/tinode/ios/commit/f8cb428951ef5b49f24db64726f7f1966c36e996

LMK if you see it again.

or-else commented 1 year ago

@aforge I think there is more to it. The server indeed did not return the token when it should have. Please take a look at the log in the attached picture. It seems like Acc.Login was treated as if it was false. I suspect @damoncoo is running a modified server.

or-else commented 1 year ago

@damoncoo What do you use for a server?

damoncoo commented 1 year ago

@damoncoo What do you use for a server?

I am using web.tinode.co, also tried api.tinode.co, same issue

damoncoo commented 1 year ago

I changed the email with different one, it worked.

I think it is the email issue, the email I used to sign up is already registered with another user. When I sign up with different user id but same email, it gives the error. Since the email is unique, it should not reply with successful message.

or-else commented 1 year ago

Thanks. I was able to reproduce server-side error.

or-else commented 1 year ago

I believe it's completely fixed.