New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Field validation in e-mail setup #1340

Closed
schakrava opened this Issue Jun 7, 2016 · 1 comment

Comments

Projects
None yet
2 participants
@schakrava
Member

schakrava commented Jun 7, 2016

Here's an e-mail bug report

Hi,

when entering a wrong formated relayhost adress fort the email alerts, the email alert functionality can become unconfigurable over the web ui.

i accidently copied a relayhost with leading blanks. This resulted in an entry in the main.cf of postfix that blocks further config tries by the web ui.

If the malformed relayhost is corrected with SSH, everything works fine again.

I suggest a better validation of this field.

Haven't found any issues regarding this.

@schakrava schakrava added the bug label Jun 7, 2016

@schakrava schakrava added this to the Pinnacles milestone Jun 7, 2016

@schakrava schakrava self-assigned this Jun 29, 2016

MFlyer added a commit to MFlyer/rockstor-jslibs that referenced this issue Jul 17, 2016

rockstor/rockstor-core#1340 Added jQuery Validation plugin extra library
Added jQuery Validation Plugin additional library
to allow new rules like pattern

MFlyer added a commit to MFlyer/rockstor-core that referenced this issue Jul 17, 2016

MFlyer added a commit to MFlyer/rockstor-core that referenced this issue Jul 17, 2016

fixes #1340 - smtp server checks
Added new jQuery validation plugin rule of type pattern
Checks for smtp server to be valid (no spaces, no starting with -,
no chars !=digits or letters, all lowercase)
@MFlyer

This comment has been minimized.

Show comment
Hide comment
@MFlyer

MFlyer Jul 17, 2016

Member

To @schakrava, not told on these 2 PR : obviously I had some tests before PR eheh :)

Member

MFlyer commented Jul 17, 2016

To @schakrava, not told on these 2 PR : obviously I had some tests before PR eheh :)

schakrava added a commit to rockstor/rockstor-jslibs that referenced this issue Jul 19, 2016

MFlyer added a commit to MFlyer/rockstor-core that referenced this issue Jul 26, 2016

@schakrava schakrava closed this in 16c1a7a Jul 26, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment