Project

Profile

Help

Actions
Send by e-mail Copy link

Issue #4313

closed

Username is mutable and can be updated/changed via REST API

Added by ttereshc over 5 years ago. Updated about 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
ttereshc
Category:
-
Sprint/Milestone:
3.0.0
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix
Sprint:
Sprint 47
Quarter:

Description

Username is mutable and can be updated/changed via REST API.
Is it on purpose? Do we rely anywhere on username?

Suggestion: make it immutable for now

Related issues

Related to Pulp - Task #2635: Determine which fields are mutable on each resource.CLOSED - COMPLETEttereshc

Actions
Related to Pulp - Task #4398: Remove "User" REST API callsCLOSED - CURRENTRELEASEipanova@redhat.com

Actions
Copied to Pulp - Test #4360: Username is mutable and can be updated/changed via REST APICLOSED - COMPLETEActions
Actions
Copy link
 #1

Updated by CodeHeeler over 5 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 47
  • Tags Easy Fix, Pulp 3 RC Blocker added
Actions
Copy link
 #2

Updated by amacdona@redhat.com over 5 years ago

Since we don't really have a full design for how users should work, I think we should remove the create & update mixins from the viewset, leaving a single "admin" user that is immutable.

This isn't at all what we want eventually, but it puts us in a great position to do whatever we need to in a backwards compatible way.

Actions
Copy link
 #3

Updated by ttereshc over 5 years ago

  • Related to Task #2635: Determine which fields are mutable on each resource. added
Actions
Copy link
 #4

Updated by ttereshc over 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ttereshc
Actions
Copy link
 #5

Updated by ttereshc over 5 years ago

  • Status changed from ASSIGNED to POST

https://github.com/pulp/pulp/pull/3829

@amacdona, I like the idea of disallowing user creation/update at this point.

Added by ttereshc over 5 years ago

Revision 17f50802 | View on GitHub

Disallow user creation/update

One admin user will exist until the full support for user management is added

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

Added by ttereshc over 5 years ago

Revision 17f50802 | View on GitHub

Disallow user creation/update

One admin user will exist until the full support for user management is added

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

Actions
Copy link
 #6

Updated by ttereshc over 5 years ago

  • Status changed from POST to MODIFIED
Actions
Copy link
 #7

Updated by bherring about 5 years ago

  • Copied to Test #4360: Username is mutable and can be updated/changed via REST API added
Actions
Copy link
 #8

Updated by kersom about 5 years ago

  • Related to Task #4398: Remove "User" REST API calls added
Actions
Copy link
 #9

Updated by daviddavis almost 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions
Copy link
 #10

Updated by bmbouter almost 5 years ago

  • Tags deleted (Pulp 3, Pulp 3 RC Blocker)
Actions
Copy link
 #11

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
Send by e-mail Copy link

Also available in: Atom PDF