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

tctl broken with PAM enabled #12547

Closed
cgoubert opened this issue May 10, 2022 · 10 comments · Fixed by #12570
Closed

tctl broken with PAM enabled #12547

cgoubert opened this issue May 10, 2022 · 10 comments · Fixed by #12570
Labels

Comments

@cgoubert
Copy link

cgoubert commented May 10, 2022

Expected behavior:

$ tctl version
Teleport v9.1.1 git:v9.1.1-0-gb0129ff5e go1.17.9
$ sudo tctl status
Cluster  teleport-staging
Version  9.2.1
Host CA  never updated
User CA  never updated
Jwt CA   never updated
CA pin   sha256:
$ sudo tctl users add test --roles=ssh
User "test" has been created but requires a password. Share this URL with the user to complete user setup, link is valid for 1h:

Current behavior:

$ tctl version
Teleport v9.2.1 git:v9.2.1-0-gc6daeda62 go1.17.9
$ sudo tctl status
ERROR: Unable to start Teleport: PAM was enabled in file configuration but this 
Teleport binary was built without PAM support. To continue either download a 
Teleport binary build with PAM support from https://goteleport.com/teleport 
or disable PAM in file configuration.
$ sudo tctl users add test --roles=ssh
ERROR: Unable to start Teleport: PAM was enabled in file configuration but this 
Teleport binary was built without PAM support. To continue either download a 
Teleport binary build with PAM support from https://goteleport.com/teleport 
or disable PAM in file configuration.

When removing PAM configuration from teleport.yaml

$ sudo tctl status
Cluster  teleport-staging
Version  9.2.1
Host CA  never updated
User CA  never updated
Jwt CA   never updated
CA pin   sha256:

Bug details:

  • Teleport v9.2.1 git:v9.2.1-0-gc6daeda62 go1.17.9
  • Installed from debian package
  • Only on tctl, PAM motd display via tsh ssh login works
  • Connecting from tctl version 9.1.1 to a teleport auth server version 9.2.1 works, the reverse doesn't
  • Debug logs say nothing interesting
  • It is impossible to downgrade as the debian repository only makes the latest release available
@cgoubert cgoubert added the bug label May 10, 2022
@cgoubert cgoubert changed the title tctl auth broken with PAM enabled tctl broken with PAM enabled May 10, 2022
@strideynet
Copy link
Contributor

strideynet commented May 11, 2022

This issue looks related to this PR: #11666

I have been able to reproduce this issue locally.

@strideynet
Copy link
Contributor

For users with this issue, the latest working version is 9.1.2

@cgoubert
Copy link
Author

Regarding leaving the former releases up on the apt repository, should I make a separate issue? Having the option to deploy a known-working fixed version via packages would be nice.

@strideynet
Copy link
Contributor

Regarding leaving the former releases up on the apt repository, should I make a separate issue? Having the option to deploy a known-working fixed version via packages would be nice.

Yeah, feel free to open a separate feature request for that and it's something we might be able to look into in the future.

@ravicious
Copy link
Member

@cgoubert Yes, please make a separate issue, this would be very helpful.

I did a quick search to see if we don't have an issue for that already, but I couldn't find anything. These two issues might be related to the fact that only the most recent version is available through apt:

@strideynet
Copy link
Contributor

Actually, it looks like this may be included as part of work on #9770 @ravicious / @cgoubert

@strideynet
Copy link
Contributor

Once this backport (#12572) goes in, we will cut a new release sometime soon in which this should be fixed.

@cgoubert
Copy link
Author

Actually, it looks like this may be included as part of work on #9770 @ravicious / @cgoubert

Yes, it looks like it, and issues like mine have already been linked. I'll subscribe to it, thanks.

@ravicious
Copy link
Member

9.2.3 is out, it should fix the issue with PAM.

@cgoubert
Copy link
Author

Confirm, issue is fixed. Thanks all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants