Project

Profile

Help

Task #2816

Rename importer auth fields

Added by amacdona@redhat.com over 3 years ago. Updated about 1 year ago.

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

100%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:

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 MVPCLOSED - CURRENTRELEASE

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

History

#1 Updated by bmbouter over 3 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 3 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 3 years ago

  • Description updated (diff)

#4 Updated by mhrivnak over 3 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 3 years ago

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

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

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

#7 Updated by amacdona@redhat.com over 3 years ago

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

#8 Updated by amacdona@redhat.com over 3 years ago

  • Status changed from ASSIGNED to POST

#9 Updated by amacdona@redhat.com over 3 years ago

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

#10 Updated by daviddavis almost 2 years ago

  • Sprint/Milestone set to 3.0.0

#11 Updated by bmbouter almost 2 years ago

  • Tags deleted (Pulp 3)

#12 Updated by bmbouter about 1 year ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF