Bug 279426

Summary: gdm: DISPLAYMANAGER_PASSWORD_LESS_LOGIN=no is ignored
Product: [openSUSE] openSUSE 10.3 Reporter: Stanislav Brabec <sbrabec>
Component: GNOMEAssignee: Stanislav Brabec <sbrabec>
Status: RESOLVED FIXED QA Contact: E-mail List <qa-bugs>
Severity: Critical    
Priority: P5 - None    
Version: Alpha 4plus   
Target Milestone: ---   
Hardware: Other   
OS: Other   
Whiteboard:
Found By: Other Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Bug Depends on:    
Bug Blocks: 192400    

Description Stanislav Brabec 2007-05-31 09:01:49 UTC
If /etc/sysconfig/displaymanager variable DISPLAYMANAGER_PASSWORD_LESS_LOGIN has value "no", password is not required.
Comment 1 Joshua Grant 2007-07-09 21:21:57 UTC
This may be related to this bug: With Alpha5plus DISPLAYMANAGER_AD_INTEGRATION="yes" appears to be ignored as I have no combobox. 
Comment 2 JP Rosevear 2007-07-31 17:23:10 UTC
Lots of commented out patches.
Comment 3 JP Rosevear 2007-08-07 19:08:36 UTC
What gdm config setting is DISPLAYMANAGER_PASSWORD_LESS_LOGIN suppose to alter Stanislav?
Comment 4 Stanislav Brabec 2007-08-08 11:52:07 UTC
None.

It should alter /etc/pam.d/gdm-autologin.

But I don't see this section in SuSEconfig.gdm.
Comment 5 Stanislav Brabec 2007-08-08 13:51:39 UTC
My fault, while fixing bug 213037 I accidentally removed this code.

Reverting and updating for current gdm-autologin contents.

Bug needs test and close. Then propose and test changes proposed in bug 192400.
Comment 6 Stanislav Brabec 2007-08-09 12:39:24 UTC
Done and tested.

Thu Aug  9 14:33:04 CEST 2007 - sbrabec@suse.cz

- Re-added DISPLAYMANAGER_PASSWORD_LESS_LOGIN to SuSEconfig.gdm
  (#279426).
- Allow password less login for both autologin and standard login.

Remaining problem in /sbin/conf.d/SuSEconfig.gdm:
If gdm is running and no session is active, UPDATE_CONFIG command is executed too early. It causes need for gdm restart to re-read configuration.
Comment 7 Joshua Grant 2007-08-09 17:59:35 UTC
Will this fix address the problem in comment #1 or do I need to open another bug?
Comment 8 JP Rosevear 2007-08-09 19:53:12 UTC
the domain combo box patch was disabled until recently, so that part should be covered.  If its not with beta 1, open a new bug.
Comment 9 JP Rosevear 2007-08-09 19:53:19 UTC
*** Bug 297625 has been marked as a duplicate of this bug. ***
Comment 10 Stanislav Brabec 2007-08-10 09:36:21 UTC
My fix not, but this one probably yes. If you will see this ChangeLog entry in your package and it will be still broken, please open a new bug:

Sat Aug  4 04:17:49 CEST 2007 - hpj@suse.de

- Update to version 2.19.5.
- Fix up gdm-conf.patch.
- Fix up gdm-2.19.3-reset-pam.patch.
- Fix up gdm-2.19.3-token-login.patch.
- Fix up and enable gdm-language-dialog-size.patch.
- Fix up and enable gdm-2.8.0.7-bg-4.patch.
- Fix up and enable gdm-2.8.0.7-domain-entry.patch.
- Fix up and enable gdm-2.8.0.7-halt-needs-root.patch.
- Fix up and enable gdm-server-timeout.patch.
- Fix up and enable gdm-2.8.0.7-some-info-in-dialog.patch.
- Fix up and enable gdm-2.17.7-vt-fallback.patch.
- Fix up and enable gdm-2.8.0.0-any-hostname-auth.patch.
- Remove gdm-2.19.3.diff (upstreamed).
- Remove gdm-2.13.0.4-audit-login.patch (upstreamed).
- Remove gdm-logdir.patch (fixed upstream).
- Remove gdm-bash-profile.patch (obsolete).