Project

Profile

Help

Task #2816

Rename importer auth fields

Added by amacdona@redhat.com over 2 years ago. Updated 6 months ago.

Status:
MODIFIED
Priority:
Normal
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
No
Sprint Candidate:
No
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:

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


Related issues

Related to Pulp - Task #2818: Align existing code base with importer attributes from the MVP MODIFIED Actions

History

#1 Updated by bmbouter over 2 years ago

  • 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.

#2 Updated by bmbouter over 2 years ago

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.

#3 Updated by bmbouter over 2 years ago

  • Description updated (diff)

#4 Updated by mhrivnak over 2 years ago

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.

#5 Updated by dkliban@redhat.com over 2 years ago

"username" and p"password" are most appropriate.

#6 Updated by amacdona@redhat.com over 2 years ago

  • Related to Task #2818: Align existing code base with importer attributes from the MVP added

#7 Updated by amacdona@redhat.com about 2 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to amacdona@redhat.com

#8 Updated by amacdona@redhat.com about 2 years ago

  • Status changed from ASSIGNED to POST

#9 Updated by amacdona@redhat.com about 2 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#10 Updated by daviddavis 6 months ago

  • Sprint/Milestone set to 3.0

#11 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3)

Please register to edit this issue

Also available in: Atom PDF