Bug 1227645 - SMTP authentication configuration changes from login to plain issues
Summary: SMTP authentication configuration changes from login to plain issues
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.4.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.5.0
Assignee: Martin Hradil
QA Contact: Milan Falešník
URL:
Whiteboard:
Depends On:
Blocks: 1255057
TreeView+ depends on / blocked
 
Reported: 2015-06-03 08:31 UTC by Eduardo Minguez
Modified: 2015-12-08 13:15 UTC (History)
5 users (show)

Fixed In Version: 5.5.0.1
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1255057 (view as bug list)
Environment:
Last Closed: 2015-12-08 13:15:18 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Reproducer video (5.59 MB, application/ogg)
2015-06-03 08:31 UTC, Eduardo Minguez
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2015:2551 0 normal SHIPPED_LIVE Moderate: CFME 5.5.0 bug fixes and enhancement update 2015-12-08 17:58:09 UTC

Description Eduardo Minguez 2015-06-03 08:31:27 UTC
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:

Comment 5 CFME Bot 2015-06-24 20:10:13 UTC
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(+)

Comment 6 CFME Bot 2015-08-19 23:01:59 UTC
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(+)

Comment 7 CFME Bot 2015-08-19 23:02:14 UTC
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(+)

Comment 8 CFME Bot 2015-08-20 00:31:36 UTC
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(+)

Comment 9 CFME Bot 2015-08-20 00:31:51 UTC
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(+)

Comment 10 CFME Bot 2015-08-20 01:48:26 UTC
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(+)

Comment 11 CFME Bot 2015-08-20 01:48:38 UTC
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(+)

Comment 12 CFME Bot 2015-08-20 02:55:21 UTC
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(+)

Comment 13 CFME Bot 2015-08-20 02:55:33 UTC
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(+)

Comment 14 Milan Falešník 2015-09-18 08:17:54 UTC
Checked in 5.5.0.1-alpha1 build that the password field together with the username field are disabled as expected.

Comment 16 errata-xmlrpc 2015-12-08 13:15:18 UTC
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


Note You need to log in before you can comment on or make changes to this bug.