pulp-cli 0.41 break username password auth in older configs (cert and key present but empty)
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 70/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- python
- Domain
- authentication, cli
Research direction
Start by reading the CLI configuration handling for cert and key, then compare how empty values from the shown TOML config are handled between versions 0.39 and 0.41. Reproduce with pulp user list using a config containing cert = "" and key = ""; done when the command still authenticates successfully with that older config.
Written by the indexing model from the issue text.
Description
Summary
Starting in v0.41 of pulp-cli, empty cert and key break username password authentication
Steps to reproduce
- ensure in
~/.config/pulp/cli.toml, for instance by generating with pulp-cli version 0.37.2
[cli]
(...)
username = "admin"
password = "xxx"
verify_ssl = true
cert = ""
key = ""
-
install pulp-cli 0.39
-
verify
pulp user liststill works -
update pulp-cli from 0.39 to 0.41
Expected behavior
pulp user list still works
Stacktrace/Error log
Www-Authenticate: Basic realm="api"
X-Content-Type-Options: nosniff
X-Frame-Options: DENY
b'{"detail":"Authentication credentials were not provided."}'
Error: Authentication failed for users_list.
Pulp and pulp-cli version info
Pulp3 Command Line Interface, Version 0.41.0
Plugin Versions:
common: 0.41.0
Additional context
A config created by pulp-cli 0.37.2 has the empty string values for cert and key
- Dominant language
- Python
- Stars
- 43
- Forks
- 52
- Avg merge
- 1d 7h
- Merged PRs (30d)
- 10
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from pulp/pulp-cli
-
bug Triage-Needed
Difficulty 2/5 1-3 hours Newbie friendliness 58/100
pulp/pulp-cli#1429 · 2 comments ·
Maintainers usually reply within 1 day
-
Difficulty 3/5 1-2 days Newbie friendliness 45/100
Maintainers usually reply within 1 day
-
feature request
Difficulty 4/5 3-5 days Newbie friendliness 35/100
pulp/pulp-cli#1210 · 1 comment ·
Maintainers usually reply within 1 day
-
container repository sync mirror option missingMay be free again A pull request for this issue was closed without being merged. Openeasy fix feature request
Difficulty 3/5 1-2 days Newbie friendliness 25/100
pulp/pulp-cli#1206 · 1 comment ·
Maintainers usually reply within 1 day
-
bug Triage-Needed
Difficulty 4/5 3-5 days Newbie friendliness 42/100
pulp/pulp-cli#1199 · 1 comment ·
Maintainers usually reply within 1 day
Similar issues
-
python triage
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
microsoft/semantic-kernel#14590 ·
Maintainers usually reply within 3 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
pinterest/pymemcache#662 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 Half a day Newbie friendliness 62/100
gnocchixyz/gnocchi#1527 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 64/100
tenstorrent/tt-model-manager#181 ·
Maintainers usually reply within 1 day