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

Salt-key using external auth and token file is broken #49222

Closed
msciciel opened this issue Aug 21, 2018 · 1 comment
Closed

Salt-key using external auth and token file is broken #49222

msciciel opened this issue Aug 21, 2018 · 1 comment
Labels
Bug broken, incorrect, or confusing behavior P4 Priority 4 severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Milestone

Comments

@msciciel
Copy link
Contributor

Description of Issue/Question

It's not possible to use token file with external auth:

$ salt-key -T --auth=pam
username: example
password:
Error: 'token_file'

$ ls salt_token
ls: cannot access salt_token: No such file or directory

### Setup
Enable external auth on salt-master:

external_auth:
pam:
users%:
- '.*'
- '@wheel'
- '@runner'
- '@jobs'
limited%:
- 'test.ping'
- 'state.apply'
- 'state.highstate'
- 'grains.get'
- 'grains.item'
- 'grains.items'
- 'pillar.get'
- 'pillar.item'
- 'pillar.items'
- 'pkg.list_pkgs'
- 'pkg.list_repos'
- 'pkg.version'


### Steps to Reproduce Issue
$ salt-key -T --auth=pam

### Versions Report

Salt Version:
Salt: 2017.7.6

Dependency Versions:
cffi: Not Installed
cherrypy: unknown
dateutil: Not Installed
docker-py: Not Installed
gitdb: Not Installed
gitpython: Not Installed
ioflo: Not Installed
Jinja2: 2.8.1
libgit2: Not Installed
libnacl: Not Installed
M2Crypto: Not Installed
Mako: Not Installed
msgpack-pure: Not Installed
msgpack-python: 0.4.6
mysql-python: Not Installed
pycparser: Not Installed
pycrypto: 2.6.1
pycryptodome: Not Installed
pygit2: Not Installed
Python: 2.7.14 (default, Jan 31 2018, 02:12:13)
python-gnupg: Not Installed
PyYAML: 3.11
PyZMQ: 14.5.0
RAET: Not Installed
smmap: Not Installed
timelib: Not Installed
Tornado: 4.2.1
ZMQ: 4.0.5

System Versions:
dist: centos 6.9 Final
locale: UTF-8
machine: x86_64
release: 2.6.32-696.30.1.el6.x86_64
system: Linux
version: CentOS 6.9 Final

@Ch3LL
Copy link
Contributor

Ch3LL commented Aug 21, 2018

thanks for the PR! We really appreciate it :) since the PR resolves the issue I will close here and we can keep the discussion in the PR. Let me know if you want me to re-open here though :)

@Ch3LL Ch3LL closed this as completed Aug 21, 2018
@Ch3LL Ch3LL added Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around P4 Priority 4 labels Aug 21, 2018
@Ch3LL Ch3LL added this to the Approved milestone Aug 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior P4 Priority 4 severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Projects
None yet
Development

No branches or pull requests

2 participants