Issue #8006
closed
pulp should treat empty string in username and password as null value
Description
when setting a proxy pulp should treat empty string in username and password as null value
Thoughts off the top of my head:
First - if you don't have a username or password - don't set them, or set them to 'nil'. Setting them to empty-string is choosing to set them to something, expecting Pulp2 to "know better" feels like a recipe for edge cases.
Second - "treat empty string as null", especially for passwords, makes Pulp2 make assumptions about the behavior of the login-system on the proxy. If that system sees null-pwd and empty-string-pwd as two different choices, then we have just created another problem, that doesn't have a workaround.
My first take on this request is to close NOTABUG. Def needs input from more ppl tho.
- Triaged changed from No to Yes
- Platform Release set to 2.21.5
- Status changed from NEW to CLOSED - WONTFIX
Turned out that this trivial fix would require to touch code for multiple plugins, given the risk of introducing regressions in multiple plugins i would rather let the current code as is and treat empty password only if a null value was provided.
- Platform Release deleted (
2.21.5)
Also available in: Atom
PDF
.