Task #2816
closed
Rename importer auth fields
Status:
CLOSED - CURRENTRELEASE
Description
basic_auth_username and basic_auth_password are useful for more than just basic auth. Rename those fields to reflect.
Some ideas:
remote_username
username
- Description updated (diff)
I think we should use username and password. I think remote_username is useful for disambiguating a remote versus local_username, which we don't have.
Actually we do have a proxy_pass which also takes a username and password embedded in the url. Perhaps using remote_username and remote_password would be good. I'd like to hear what others think.
- Description updated (diff)
I like the simplicity of "username" and "password". I don't think anyone will be confused about what they mean, especially given a succinct line of documentation.
"username" and p"password" are most appropriate.
- Related to Task #2818: Align existing code base with importer attributes from the MVP added
- Status changed from NEW to ASSIGNED
- Assignee set to amacdona@redhat.com
- Status changed from ASSIGNED to POST
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
- Sprint/Milestone set to 3.0.0
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Also available in: Atom
PDF
.
Align Importer fields and text with MVP document
closes #2818 re #2817 closes #2816
https://pulp.plan.io/projects/pulp/wiki/Pulp_3_Minimum_Viable_Product