Project

Profile

Help

Issue #3846

closed

Wrong status code

Added by vdusek over 5 years ago. Updated almost 4 years ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 42
Quarter:

Description

When I want to create a repository with a name which is already used for another repository, it returns status code 400 - Bad Request. According to this topic HTTP response code for POST when resource already exists I would suggest use 409 - Conflict.

Actions #1

Updated by CodeHeeler over 5 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 39
Actions #2

Updated by daviddavis over 5 years ago

FYI, there are a bunch of other models that have similar unique constraints: publishers, remotes, exporters, distributions, users, artifacts.

Actions #3

Updated by vdusek over 5 years ago

According to the django-rest-framework issue tracker on Github (4509, 5385, ...) it looks like we're not the first to deal with this problem. If I understand it right, collaborators don't have any interest in tackling it themselves or their point of view is that 400 is OK for uniqueness constraints. So I guess we'll have to deal with it by implementing our own subclass of ApiException.

Actions #4

Updated by vdusek over 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to vdusek
Actions #5

Updated by dkliban@redhat.com over 5 years ago

  • Sprint changed from Sprint 39 to Sprint 40
Actions #6

Updated by vdusek over 5 years ago

  • Status changed from ASSIGNED to POST
Actions #7

Updated by vdusek over 5 years ago

PR: 3569

Actions #8

Updated by rchan over 5 years ago

  • Sprint changed from Sprint 40 to Sprint 41
Actions #9

Updated by rchan over 5 years ago

  • Sprint changed from Sprint 41 to Sprint 42
Actions #10

Updated by vdusek over 5 years ago

  • Status changed from POST to CLOSED - WONTFIX

Based on the reaction of upstream and discussion on the PR.

Actions #11

Updated by daviddavis almost 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #12

Updated by bmbouter almost 5 years ago

  • Tags deleted (Pulp 3)
Actions #13

Updated by bmbouter almost 4 years ago

  • Category deleted (14)

We are removing the 'API' category per open floor discussion June 16, 2020.

Also available in: Atom PDF