Bug 1045

Summary: Missing option for ignoring the /etc/nologin file
Product: Portable OpenSSH Reporter: Tomas Mraz <t8m>
Component: PAM supportAssignee: Assigned to nobody <unassigned-bugs>
Status: CLOSED FIXED    
Severity: enhancement CC: ssh-bugs
Priority: P2 Keywords: needs-release-note
Version: 4.0p1   
Hardware: All   
OS: Linux   
Bug Depends on:    
Bug Blocks: 1047    
Attachments:
Description Flags
Proposed patch
none
Skip nologin check if PAM is enabled. djm: ok+

Description Tomas Mraz 2005-05-25 23:47:13 AEST
On some systems /etc/nologin file can be processed by a PAM module or it can be
meant only for local console logins. sshd is missing an option to ignore this file.
Comment 1 Tomas Mraz 2005-05-25 23:49:13 AEST
Created attachment 922 [details]
Proposed patch

This patch adds a new IgnoreNologin option which implements this enhancement.
Comment 2 Darren Tucker 2005-05-29 12:50:40 AEST
I'm not sure sshd needs another knob for this.

Perhaps sshd should ignore /etc/nologin and leave it to the PAM modules.  The
session module could then do whatever it wants WRT the source of the login.  I
dunno.
Comment 3 Tomas Mraz 2005-05-30 05:48:15 AEST
I would be happy enough with this solution too -> simply switch off nologin
checking if PAM is used.
Comment 4 Damien Miller 2005-06-03 12:50:25 AEST
I think that ignoring nologin for the PAM case is a good idea, but it is a
change that we will have to publicise.
Comment 5 Darren Tucker 2005-10-03 19:55:05 AEST
Created attachment 981 [details]
Skip nologin check if PAM is enabled.
Comment 6 Damien Miller 2005-10-30 15:09:21 AEDT
Comment on attachment 981 [details]
Skip nologin check if PAM is enabled.

ok by me (with release note). maybe we need a README.PAM?
Comment 7 Darren Tucker 2005-10-30 15:34:41 AEDT
Applied #981, thanks.  (I added a needs-release-note keyword).

Rather than yet another README, I think we should just add a PAM section to README.platform.
Comment 8 Darren Tucker 2006-10-07 11:40:23 AEST
Change all RESOLVED bug to CLOSED with the exception of the ones fixed post-4.4.