🐞 [Bug]: Change mail config validator on taiga

[A-Harby]

Is there an existing issue for this?

• I have searched the existing issues

which package/s did you face the problem with?

Dashboard

What happened?

The same issue is happing to the taiga, and also check all of the other application stmp.

Steps To Reproduce

No response

which network/s did you face the problem on?

Dev

version

448a213

Twin ID/s

No response

Node ID/s

No response

Farm ID/s

No response

Contract ID/s

No response

Relevant screenshots/screen records

#2152 (comment)

Relevant log output

.

[amiraabouhadid]

Use validation used in discourse solution. Remove the condition applying validation to just discourse.
Check after the validation change that the solution won't crash because other solution might inherently have different email requirements.
Check if you can still send emails with SMTP

[samaradel]

Work in Progress (WIP):

utils created to validate SMTP (email, apiKey, username), faced an error .. investigating..

[samaradel]

Work Completed:

• Remove isDiscourse check and apply adding email or username for all apps with SMTP.
• Create a function to validate applications with SMTP.
• All solutions with SMTP are tested except Taiga which is blocked on Taiga: Email not working  tf-images#293

[samaradel]

Work Completed:

• Doc string added to explain the new IsAlphaExpectDashAndUnderscore validator function.
• Unit test added too.

[khaledyoussef24]

verified on devnet
Version a78daae

now user names are allowed so user can use api keys or user name normally

passwords is allowed more than 50 characters
validation is also added (in this case i used space in the user name to test validation ):

here to check that passwords can be more than 50 characters :

here to check the validation of passwords when inserting space :

testcase

[khaledyoussef24]

after deploying 2 taigas did not recieve any mails on the mail I used in the setup is this related ?

[samaradel]

we mentioned here that taiga part is blocked on threefoldtech/tf-images#293

[khaledyoussef24]

Okay, thanks for clarifying.