search

tigase / siskin-im

(M) Public Project of Tigase Messenger for iOS devices based on Tigase Swift XMPP library.
GNU General Public License v3.0
175 stars 37 forks source link

Registering account don't work #150

Open pravi opened 3 years ago

pravi commented 3 years ago

Describe the bug Registering account does not work.

To Reproduce Steps to reproduce the behavior:

  1. install app
  2. Open app
  3. Try to register
  4. It does not work, save settings has no effect and adding contact says account not online.

Expected behavior We should be able to register accounts

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

Details (please complete the following information):

Additional context Add any other context about the problem here.

licaon-kter commented 3 years ago

Which server? The Tigase ones or others?

hantu85 commented 3 years ago

There are no details here. Please add missing details:

I can say that I've tested it with a few servers and it worked without any issue.

pravi commented 3 years ago

iPhone 11 Pro and iPhone 12. sure.im and tigase.im was tried.

pravi commented 3 years ago

Installed from app store.

pravi commented 3 years ago

It could be a non intuitive flow as well.

hantu85 commented 3 years ago

Have you read the account registration instructions presented to you? You need to confirm your email by tapping on the link sent to your email address. Then you need to enable accounts in the SiskinIM settings to make them work.

pravi commented 3 years ago

3 of my friends could not sign up, I tried myself as well.

pravi commented 3 years ago

But this was not clear at all from the ui that email confirmation is necessary.

pravi commented 3 years ago

Probably there should be a standard way to represent pending confirmation status from server and display it on the client side.

theotheroracle commented 2 years ago

can confirm this is an issue, was able to register in monal no issue, for example to conversations.im but it did not work here, unending loading spinner . iphone 7 ios 15.4

hantu85 commented 2 years ago

Unfortunately, there is no way to indicate that the account requires confirmation (in the XMPP specification it is not defined). We will work on improving UX in this case.

pravi commented 2 years ago

@hantu85 I can see conversations on android asks for a captcha confirmation for conversations.im accounts. May be we can do a similar thing to ask for the confirmation code sent via email or possibly adding sms otp confirmation as another option - as many people seem to prefer sms over email?

pravi commented 2 years ago

I think https://xmpp.org/extensions/xep-0158.html is flexible enough to authenticate via email confirmation.

hantu85 commented 2 years ago

The confirmation to create an account (captcha support) is already implemented (and even used by the server during account creation). The issue here is to confirm the email account - the server sends a link and the user needs to open it but the user is not aware of that. We plan to fix that but other things had higher priority.

pravi commented 2 years ago

Can't the email confirmation also be presented as a second challenge so user will know they have to respond? The clicking link can be replaced by entering a confirmation code received in email. But if you think you already found a better way, that should be fine.

hantu85 commented 2 years ago

I think about using SASL2 for that. It is implemented in a preview version and hopefully, we will be able to use it soon (waiting for publication of updated version of the XEP).

pravi commented 1 year ago

What do you think about XEP-0389 in this context?

pravi commented 1 year ago

@hantu85 I think sasl2 update is published now https://xmpp.org/extensions/xep-0388.html

Neustradamus commented 11 months ago

There are:

arthef commented 11 months ago

Hi, thank you for updating the issue.

Just so you know. We have migrated onto Tygrys platform. All our projects are available at https://tigase.dev/ address and the Tigase SiskinIM project is here: https://tigase.dev/tigase/_clients/siskin-im

All the existing issues have been migrated as well. This particular issue can be found here: https://tigase.dev/tigase/_clients/siskin-im/~issues/522