Project

Profile

Help

Story #2984

As an API user, I get user friendly feedback when I use an invalid value

Added by daviddavis@redhat.com about 1 month ago. Updated 5 days ago.

Status:
NEW
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
% Done:

0%

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

Description

There are several places in Pulp 3 where we ought to be using validation but are not. Here are a few examples with reproducers:

https://pulp.plan.io/issues/2974
https://pulp.plan.io/issues/2975
https://pulp.plan.io/issues/2976

There may be more issues too. Instead of opening individual bugs/tasks, this story is go through the API and add validation for any existing fields that need it.


Checklist


Related issues

Related to Pulp - Issue #2976: 500 error when I try to create a username with more than 150 chars CLOSED - DUPLICATE Actions
Duplicated by Pulp - Issue #2975: Username field doesn't validate characters CLOSED - DUPLICATE Actions
Duplicated by Pulp - Issue #2974: Error when attempting to create two repos with the same name CLOSED - DUPLICATE Actions

History

#1 Updated by daviddavis@redhat.com about 1 month ago

  • Duplicated by Issue #2975: Username field doesn't validate characters added

#2 Updated by daviddavis@redhat.com about 1 month ago

  • Related to Issue #2976: 500 error when I try to create a username with more than 150 chars added

#3 Updated by daviddavis@redhat.com about 1 month ago

  • Duplicated by Issue #2974: Error when attempting to create two repos with the same name added

#4 Updated by daviddavis@redhat.com about 1 month ago

  • Tags Pulp 3 added

#5 Updated by amacdona@redhat.com 5 days ago

  • Groomed changed from No to Yes
  • Sprint Candidate changed from No to Yes

I am seeing a 500 for duplicates across the board. The serializers do not check for uniqueness and the db error bubbles up.

#6 Updated by mhrivnak 5 days ago

  • Sprint/Milestone set to Sprint 26

Please register to edit this issue

Also available in: Atom PDF