Patch for using /etc/issue file as a login banner about shellinabox

[GoogleCodeExporter]

I need to think about this a little more. I am not quite sure this is the right 
solution. PAM already has support for outputting /etc/issue. But that currently 
doesn't seem to do anything in ShellInABox. That does look like a bug.

As for the mailing list, there is an old mailing list for this project. But it 
has 
always been very low traffic, so I have stopped advertising it, when I switched 
to 
the new code base.

The issue tracker works a little better for me, as helps me keep track of what 
still 
needs to be done before the next official release.

But if you think a mailing list would be more appropriate, I could certainly 
revive 
it.

Original comment by [email protected] on 27 Aug 2009 at 6:03

• Changed state: Accepted

[GoogleCodeExporter]

I'm alright with using PAM for this if 1) it works :) and 2) I can figure out 
how to
work it, i.e. examples that I can use, as I'm not that PAM-saavy.  I'm currently
using busybox (http://www.busybox.net) for pretty much everything on the 
systems that
I'm using shellinabox with, but I have PAM support turned off inside it right 
now.

I don't really need the mailing list, however, it's not abundantly clear where 
people
can get help when they hit your main project page, and my experience has been 
that
people need things to be abundantly clear most of the time.  Right now, you 
sort of
have to click on everything and then find the issues page before you have a 
chance of
offering feedback.

Original comment by [email protected] on 27 Aug 2009 at 6:16

[GoogleCodeExporter]

On Debian systems, you can create a file /etc/pam.d/shellinabox:

  auth required pam_issue.so issue=/etc/issue
  @include other

I haven't tested this on any distribution, but I would assume things should be 
very similar.

Please note that pam_issue has a bug in many commonly shipped versions. Unless 
you pass the "noesc" option, it 
won't be able to read any issue files. Alternatively, apply this patch:

--- pam-1.1.0/modules/pam_issue/pam_issue.c.orig    2009-11-21 14:08:21.866374292 
-0800
+++ pam-1.1.0/modules/pam_issue/pam_issue.c 2009-11-21 14:31:05.086176879 -0800
@@ -277,8 +277,8 @@ read_issue_quoted(pam_handle_t *pamh, FI
        return PAM_BUF_ERR;
        }
        issue = new_issue;
+       strcat(issue, buf);
    }
-   strcat(issue, buf);
     }

     if (ferror(fp)) {

Original comment by [email protected] on 21 Nov 2009 at 10:32

• Changed state: WontFix