console icon indicating copy to clipboard operation
console copied to clipboard

Published 20 hours ago β€’ verified publisher icon appwrite

πŸ› Bug Report: Wrong Message When Changing Email on Console

Open ItzNotABug opened this issue 11 months ago β€’ 9 comments

πŸ‘Ÿ Reproduction steps

Assuming you have 2 different Email Addresses, each with an Account on Cloud.

  1. Go to second account
  2. Account > Edit Email Address
  3. Enter the Email Address of the First account > Enter password > Update
  4. Error - There was an error processing your request. Please check the inputs and try again.

The error should properly depict that the new Email Address already exists on Appwrite or connected to an Organization.

πŸ‘ Expected behavior

Show proper error message as mentioned above.

πŸ‘Ž Actual Behavior

Current shown error is not too clear regarding the actual error.

🎲 Appwrite version

Appwrite Cloud

πŸ’» Operating system

Linux

🧱 Your Environment

No response

πŸ‘€ Have you spent some time to check if this issue has been raised before?

  • [X] I checked and didn't find similar issue

🏒 Have you read the Code of Conduct?

ItzNotABug avatar Mar 17 '24 07:03 ItzNotABug

Hi there, I would like to work on this issue, please.

MatthijsDV2004 avatar Mar 22 '24 22:03 MatthijsDV2004

Nevermind this bug seems to have been fixed already - should be marked as complete / solved.

MatthijsDV2004 avatar Mar 22 '24 23:03 MatthijsDV2004

@ItzNotABug, thanks for creating this issue! πŸ™πŸΌ You mentioned:

The error should properly depict that the new Email Address already exists on Appwrite or connected to an Organization.

We actually don't want to say the email address already exists because that let's the user know someone has that email on the platform. As such, can this be closed?

stnguyen90 avatar Mar 29 '24 18:03 stnguyen90

@stnguyen90, The point for not really saying that the email already exists is valid but the current message still seems pretty vague. Even though an email might be in use, the current error says there's an error with the input [presumably frontend form].

We could instead use something or similar like -

  1. Unable to update email. Please ensure it is correct and not already in use.
  2. Unable to update email. Please ensure email address is yours and is correctly typed.

ItzNotABug avatar Mar 30 '24 06:03 ItzNotABug

@gewenyu99, what are your thoughts on the copy of the error message?

stnguyen90 avatar Apr 04 '24 19:04 stnguyen90

@stnguyen90 It's interesting because the solution LinkedIn, Facebook, etc use is actually just a captcha that blocks repeated attempts to snoop for emails.

I really want to be able to display a different response to the Console SDK vs Client/Server SDK.

For for, unable to update email is as far as we should go. Please ensure it is correct and not already in use. is basically saying it's in use. They can tell if it's a valid email they tried.

gewenyu99 avatar Apr 15 '24 22:04 gewenyu99

I really want to be able to display a different response to the Console SDK vs Client/Server SDK.

@gewenyu99, to do this, the Console needs to check the type (assuming the type is unique enough) and then show some custom message, ignoring the message from the API response.

stnguyen90 avatar Apr 16 '24 23:04 stnguyen90

@stnguyen90 Does this API exist for client? Doesn't it require a password? Maybe we just need to rate limit it？

gewenyu99 avatar Apr 17 '24 18:04 gewenyu99

Does this API exist for client? Doesn't it require a password? Maybe we just need to rate limit it？

@gewenyu99, yes, the update email endpoint requires a password. There must be a default rate limit on it.

stnguyen90 avatar Apr 17 '24 21:04 stnguyen90