Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: pam integration? #742

Closed
Kellel opened this issue Feb 1, 2017 · 3 comments
Closed

Feature: pam integration? #742

Kellel opened this issue Feb 1, 2017 · 3 comments
Assignees
Labels
feature-request Used for new features in Teleport, improvements to current should be #enhancements

Comments

@Kellel
Copy link

Kellel commented Feb 1, 2017

Initial Comment

Present motd information on login similar to OpenSSH
https://www.ietf.org/rfc/rfc4252.txt Section 5.4

Proposal

Teleport is going to (optionally) use PAM to create user sessions. This will enable the full pipeline of PAM-driven session customization, which will give us access to all PAM plugins, including motd

@kontsevoy kontsevoy changed the title Feature: Implement login banners Feature: pam integration? Feb 1, 2017
@kontsevoy kontsevoy added feature-request Used for new features in Teleport, improvements to current should be #enhancements help wanted labels Feb 1, 2017
@kontsevoy
Copy link
Contributor

kontsevoy commented Feb 1, 2017

@Kellel OpenSSH is most distros is actually configured to integrate with pam and it's pam who's displaying motd-like prompt.

Frankly we feel this will only add unneeded complexity because Teleport performs authentication on a cluster-level, i.e. individual nodes can be as dumb as they come (for example people use Teleport to SSH into docker containers randomly flying around).

The utility of pam is somewhat questionable in a clustered environment, where user identity is stored elsewhere and user authentication happens before the SSH connection is even made (at SSH bastion/proxy), see teleport login command which doesn't require a host name: in Teleport world hosts are "dumb cattle". So it's a head scratcher. Do you mind me ask: why are you asking?

But if someone is willing to look into lightweight integration with pam, we're open to accept a decent PR. :-)

I'll mark it as "help wanted".

@eiginn
Copy link

eiginn commented Feb 16, 2017

One use case we have for using pam is assigning an "auid" that is process inherited via pam_loginuid.so and we capture this in auditd logs. Allows users to sudo to root while maintaining a trail.

@russjones
Copy link
Contributor

Implemented in #1766.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Used for new features in Teleport, improvements to current should be #enhancements
Projects
None yet
Development

No branches or pull requests

5 participants