Project

Profile

Help

Issue #1058

can't update users on rhel7 and f20

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

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
3. High
Version:
Platform Release:
2.7.0
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

rpm -qa pulp-server

pulp-server-2.7.0-0.3.beta.fc20.noarch

pulp-admin -u admin -p admin auth user create --login test-user
Enter password for user [test-user] : 123456
Re-enter password for user [test-user]: 123456
User [test-user] successfully created

>> pulp-admin -u admin -p admin  auth user update --login test-user --password 654321
Invalid properties: ['p']

>> pulp-admin -u admin -p admin  auth user update --login test-user --password '654321'
Invalid properties: ['p']

>> pulp-admin -u admin -p admin  auth user update --login test-user --password -p
Invalid properties: ['p']

>> pulp-admin -u admin -p admin  auth user update --login test-user -p
Enter password for user [test-user] : <654321>
Re-enter password for user [test-user]: <654321>
Invalid properties: ['p']

>> pulp-admin -u admin -p admin  auth user update --login test-user --password 
Usage: pulp-admin [options]

Command: update
Description: changes metadata of an existing user

Available Arguments:

  --login    - (required) identifies the user to be updated
  --name     - user-readable full name of the user
  --password - new password for the user, use -p if you want to be prompted for
               the password
  -p         - if specified, you will be prompted to enter new password for the
               user
>> pulp-admin -u admin -p admin  auth user update --login test-user -p 654321
Command: update
Description: changes metadata of an existing user

Available Arguments:

  --login    - (required) identifies the user to be updated
  --name     - user-readable full name of the user
  --password - new password for the user, use -p if you want to be prompted for
               the password
  -p         - if specified, you will be prompted to enter new password for the
               user
>> pulp-admin -u admin -p admin  auth user update --login test-user -p '654321'
Command: update
Description: changes metadata of an existing user

Available Arguments:

  --login    - (required) identifies the user to be updated
  --name     - user-readable full name of the user
  --password - new password for the user, use -p if you want to be prompted for
               the password
  -p         - if specified, you will be prompted to enter new password for the
               user

>> pulp-admin -u admin -p admin  auth user update --login test-user -p 
Enter password for user [test-user] : <qazxsw>
Re-enter password for user [test-user]: <qazxsw>
Invalid properties: ['p']

However it checks if passwords match:

>> pulp-admin -u admin -p admin  auth user update --login test-user -p 
Enter password for user [test-user] : <qwerty>
Re-enter password for user [test-user]: <ytrewq>
Passwords do not match

The same output was observed on rhel7

It's also impossible to update user's name:

>> pulp-admin  auth user create --login new_user --password 654321 --name "Test_user"
User [new_user] successfully created

>> pulp-admin  auth user update --login new_user --name "BlaBla"
Invalid properties: ['p']

Related issues

Related to Pulp - Issue #1085: pulp-admin and pulp-admin auth user update both use the -p flag for different purposesCLOSED - WONTFIX<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision daa198d1 View on GitHub
Added by ipanova@redhat.com over 6 years ago

Fixed failure during pulp-admin user update.

closes #1058 https://pulp.plan.io/issues/1058

Revision daa198d1 View on GitHub
Added by ipanova@redhat.com over 6 years ago

Fixed failure during pulp-admin user update.

closes #1058 https://pulp.plan.io/issues/1058

History

#1 Updated by igulina@redhat.com over 6 years ago

  • Description updated (diff)

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

  • Subject changed from can't update user's password on rhel7 and f20 to can't update users on rhel7 and f20
  • Description updated (diff)

#4 Updated by bmbouter over 6 years ago

  • Priority changed from Normal to High
  • Severity changed from 2. Medium to 3. High
  • Platform Release set to 2.7.0
  • Triaged changed from No to Yes
  • Tags deleted (Easy Fix)

#5 Updated by ipanova@redhat.com over 6 years ago

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

#6 Updated by ipanova@redhat.com over 6 years ago

  • Status changed from ASSIGNED to POST

#7 Updated by rbarlow over 6 years ago

I'm <facepalm>ing that the -p flag can be used twice in this command - once to authenticate for the action you are performing, and again to specify the data for the action you are performing. We should file a separate issue about that.

#8 Updated by rbarlow over 6 years ago

  • Related to Issue #1085: pulp-admin and pulp-admin auth user update both use the -p flag for different purposes added

#9 Updated by ipanova@redhat.com over 6 years ago

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

#10 Updated by dkliban@redhat.com over 6 years ago

  • Status changed from MODIFIED to 5

#11 Updated by igulina@redhat.com over 6 years ago

  • Status changed from 5 to 6
>> rpm -qa pulp-server
pulp-server-2.7.0-0.4.beta.el7.noarch

>> pulp-admin -u admin -p admin  auth user update --login test-user --password 654321
User [test-user] successfully updated

>> pulp-admin -u admin -p admin  auth user update --login test-user --password '654321'
User [test-user] successfully updated

>> pulp-admin -u admin -p admin  auth user update --login test-user --password -p
User [test-user] successfully updated

>> pulp-admin login -u test-user 
Enter password: 
Successfully logged in. Session certificate will expire at Jul 17 09:37:47 2015
GMT.

>> pulp-admin login -u test-user -p -p
Successfully logged in. Session certificate will expire at Jul 17 09:38:10 2015
GMT.

>> pulp-admin -u admin -p admin  auth user update --login test-user --password 
Usage: pulp-admin [options]

Command: update
Description: changes metadata of an existing user

Available Arguments:

  --login    - (required) identifies the user to be updated
  --name     - user-readable full name of the user
  --password - new password for the user, use -p if you want to be prompted for
               the password
  -p         - if specified, you will be prompted to enter new password for the
               user
>> pulp-admin -u admin -p admin  auth user update --login test-user -p 654321
Command: update
Description: changes metadata of an existing user

Available Arguments:

  --login    - (required) identifies the user to be updated
  --name     - user-readable full name of the user
  --password - new password for the user, use -p if you want to be prompted for
               the password
  -p         - if specified, you will be prompted to enter new password for the
               user
>> pulp-admin -u admin -p admin  auth user update --login test-user -p
Enter password for user [test-user] : 
Re-enter password for user [test-user]: 
User [test-user] successfully updated

>> pulp-admin  auth user create --login test-user --password 654321 --name "Test_user" 
The specified user does not have permission to execute the given command

>> pulp-admin logout
Session certificate successfully removed.

>> pulp-admin  auth user create --login test-user --password 654321 --name "Test_user" 
The specified user does not have permission to execute the given command

>> pulp-admin login -u admin -p admin
Successfully logged in. Session certificate will expire at Jul 17 09:53:29 2015
GMT.

>> pulp-admin  auth user create --login test-user --password 654321 --name "Test_user" 
A resource with the ID "test-user" already exists.

>> pulp-admin  auth user update --login new_user --name "BlaBla" 
User [new_user] successfully updated

#12 Updated by amacdona@redhat.com almost 6 years ago

  • Status changed from 6 to CLOSED - CURRENTRELEASE

#14 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF