Bug 1227645
| Summary: | SMTP authentication configuration changes from login to plain issues | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | Eduardo Minguez <eminguez> | ||||
| Component: | UI - OPS | Assignee: | Martin Hradil <mhradil> | ||||
| Status: | CLOSED ERRATA | QA Contact: | Milan Falešník <mfalesni> | ||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | medium | ||||||
| Version: | 5.4.0 | CC: | cpelland, hkataria, jhardy, mpovolny, obarenbo | ||||
| Target Milestone: | GA | Keywords: | Reopened, ZStream | ||||
| Target Release: | 5.5.0 | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | 5.5.0.1 | Doc Type: | Bug Fix | ||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | |||||||
| : | 1255057 (view as bug list) | Environment: | |||||
| Last Closed: | 2015-12-08 13:15:18 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Bug Depends On: | |||||||
| Bug Blocks: | 1255057 | ||||||
| Attachments: |
|
||||||
New commit detected on manageiq/master: https://github.com/ManageIQ/manageiq/commit/a93db921149aced36772e0b77aa4fecbb12bcb78 commit a93db921149aced36772e0b77aa4fecbb12bcb78 Author: Martin Hradil <mhradil> AuthorDate: Wed Jun 10 13:52:16 2015 +0000 Commit: Martin Hradil <mhradil> CommitDate: Wed Jun 24 13:20:37 2015 +0000 Server Settings - SMTP disable password field iff Authentication is none Previously, only the username field got disabled, this disables the password fields as well. https://bugzilla.redhat.com/show_bug.cgi?id=1227645 vmdb/app/controllers/ops_controller/settings/common.rb | 2 ++ 1 file changed, 2 insertions(+) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=e7df59a1dc0593b5ed447e5135345be3243c076c commit e7df59a1dc0593b5ed447e5135345be3243c076c Author: Martin Hradil <mhradil> AuthorDate: Wed Jun 10 13:52:16 2015 +0000 Commit: Martin Hradil <mhradil> CommitDate: Wed Aug 19 14:34:17 2015 +0000 Server Settings - SMTP disable password field iff Authentication is none Previously, only the username field got disabled, this disables the password fields as well. https://bugzilla.redhat.com/show_bug.cgi?id=1227645 (cherry picked from commit a93db921149aced36772e0b77aa4fecbb12bcb78) vmdb/app/controllers/ops_controller/settings/common.rb | 2 ++ 1 file changed, 2 insertions(+) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=7ded04c738a0e7d59661d484b3b8cb3431525806 commit 7ded04c738a0e7d59661d484b3b8cb3431525806 Merge: 4013cf2 e7df59a Author: Dan Clarizio <dclarizi> AuthorDate: Wed Aug 19 12:12:22 2015 -0400 Commit: Dan Clarizio <dclarizi> CommitDate: Wed Aug 19 12:12:22 2015 -0400 Merge branch 'bz1255057' into '5.4.z' Server Settings - SMTP disable password field iff Authentication is none Previously, only the username field got disabled, this disables the password fields as well. https://bugzilla.redhat.com/show_bug.cgi?id=1227645 (cherry picked from commit a93db921149aced36772e0b77aa4fecbb12bcb78) 5.4 bz: https://bugzilla.redhat.com/show_bug.cgi?id=1255057 clean cherry pick See merge request !232 vmdb/app/controllers/ops_controller/settings/common.rb | 2 ++ 1 file changed, 2 insertions(+) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=e7df59a1dc0593b5ed447e5135345be3243c076c commit e7df59a1dc0593b5ed447e5135345be3243c076c Author: Martin Hradil <mhradil> AuthorDate: Wed Jun 10 13:52:16 2015 +0000 Commit: Martin Hradil <mhradil> CommitDate: Wed Aug 19 14:34:17 2015 +0000 Server Settings - SMTP disable password field iff Authentication is none Previously, only the username field got disabled, this disables the password fields as well. https://bugzilla.redhat.com/show_bug.cgi?id=1227645 (cherry picked from commit a93db921149aced36772e0b77aa4fecbb12bcb78) vmdb/app/controllers/ops_controller/settings/common.rb | 2 ++ 1 file changed, 2 insertions(+) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=7ded04c738a0e7d59661d484b3b8cb3431525806 commit 7ded04c738a0e7d59661d484b3b8cb3431525806 Merge: 4013cf2 e7df59a Author: Dan Clarizio <dclarizi> AuthorDate: Wed Aug 19 12:12:22 2015 -0400 Commit: Dan Clarizio <dclarizi> CommitDate: Wed Aug 19 12:12:22 2015 -0400 Merge branch 'bz1255057' into '5.4.z' Server Settings - SMTP disable password field iff Authentication is none Previously, only the username field got disabled, this disables the password fields as well. https://bugzilla.redhat.com/show_bug.cgi?id=1227645 (cherry picked from commit a93db921149aced36772e0b77aa4fecbb12bcb78) 5.4 bz: https://bugzilla.redhat.com/show_bug.cgi?id=1255057 clean cherry pick See merge request !232 vmdb/app/controllers/ops_controller/settings/common.rb | 2 ++ 1 file changed, 2 insertions(+) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=e7df59a1dc0593b5ed447e5135345be3243c076c commit e7df59a1dc0593b5ed447e5135345be3243c076c Author: Martin Hradil <mhradil> AuthorDate: Wed Jun 10 13:52:16 2015 +0000 Commit: Martin Hradil <mhradil> CommitDate: Wed Aug 19 14:34:17 2015 +0000 Server Settings - SMTP disable password field iff Authentication is none Previously, only the username field got disabled, this disables the password fields as well. https://bugzilla.redhat.com/show_bug.cgi?id=1227645 (cherry picked from commit a93db921149aced36772e0b77aa4fecbb12bcb78) vmdb/app/controllers/ops_controller/settings/common.rb | 2 ++ 1 file changed, 2 insertions(+) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=7ded04c738a0e7d59661d484b3b8cb3431525806 commit 7ded04c738a0e7d59661d484b3b8cb3431525806 Merge: 4013cf2 e7df59a Author: Dan Clarizio <dclarizi> AuthorDate: Wed Aug 19 12:12:22 2015 -0400 Commit: Dan Clarizio <dclarizi> CommitDate: Wed Aug 19 12:12:22 2015 -0400 Merge branch 'bz1255057' into '5.4.z' Server Settings - SMTP disable password field iff Authentication is none Previously, only the username field got disabled, this disables the password fields as well. https://bugzilla.redhat.com/show_bug.cgi?id=1227645 (cherry picked from commit a93db921149aced36772e0b77aa4fecbb12bcb78) 5.4 bz: https://bugzilla.redhat.com/show_bug.cgi?id=1255057 clean cherry pick See merge request !232 vmdb/app/controllers/ops_controller/settings/common.rb | 2 ++ 1 file changed, 2 insertions(+) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=e7df59a1dc0593b5ed447e5135345be3243c076c commit e7df59a1dc0593b5ed447e5135345be3243c076c Author: Martin Hradil <mhradil> AuthorDate: Wed Jun 10 13:52:16 2015 +0000 Commit: Martin Hradil <mhradil> CommitDate: Wed Aug 19 14:34:17 2015 +0000 Server Settings - SMTP disable password field iff Authentication is none Previously, only the username field got disabled, this disables the password fields as well. https://bugzilla.redhat.com/show_bug.cgi?id=1227645 (cherry picked from commit a93db921149aced36772e0b77aa4fecbb12bcb78) vmdb/app/controllers/ops_controller/settings/common.rb | 2 ++ 1 file changed, 2 insertions(+) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=7ded04c738a0e7d59661d484b3b8cb3431525806 commit 7ded04c738a0e7d59661d484b3b8cb3431525806 Merge: 4013cf2 e7df59a Author: Dan Clarizio <dclarizi> AuthorDate: Wed Aug 19 12:12:22 2015 -0400 Commit: Dan Clarizio <dclarizi> CommitDate: Wed Aug 19 12:12:22 2015 -0400 Merge branch 'bz1255057' into '5.4.z' Server Settings - SMTP disable password field iff Authentication is none Previously, only the username field got disabled, this disables the password fields as well. https://bugzilla.redhat.com/show_bug.cgi?id=1227645 (cherry picked from commit a93db921149aced36772e0b77aa4fecbb12bcb78) 5.4 bz: https://bugzilla.redhat.com/show_bug.cgi?id=1255057 clean cherry pick See merge request !232 vmdb/app/controllers/ops_controller/settings/common.rb | 2 ++ 1 file changed, 2 insertions(+) Checked in 5.5.0.1-alpha1 build that the password field together with the username field are disabled as expected. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2015:2551 |
Created attachment 1034177 [details] Reproducer video Description of problem: * In CF3.2 beta, if you modify the smtp configuration from login to plain, the password field is not disabled. * In CF3.2 beta, if you modify the smtp configuration from plain to login, the password field is not enabled, you need to save in order to have it enabled. Version-Release number of selected component (if applicable): 3.2 beta How reproducible: Check the video attached for a reproducer Steps to Reproduce: 1. Go to configure -> configuration 2. Modify the smtp settings from login to none and viceversa 3. Check the password field Actual results: Password field is enabled or disabled wrong Expected results: Password field is enabled or disabled when needed (plain = disabled, login = enabled) Additional info: