Project

General

Profile

Actions

Bug #8612

closed

API Delete activation key does not respond with content

Added by Christine Fouant over 9 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Category:
API
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1171092
Description of problem:
There is no output in the response on a DELETE request on activation keys.

root@li-lc-1017:/home/hoiroot# curl -K /opt/hoici/etc/curl-hoici.conf -H 'Content-Type: application/json' -d '{"per_page":9999}' -XDELETE https://localhost/katello/api/v2/activation_keys/31
root@li-lc-1017:/home/hoiroot#

Applying the following patch:

--- activation_keys_controller.rb.old 2014-12-05 10:16:25.718267583 0000
++ activation_keys_controller.rb 2014-12-05 10:16:46.512486372 0000
@ -77,7 +77,7 @
param :id, :identifier, :desc => N_("ID of the activation key"), :required => true
def destroy
sync_task(::Actions::Katello::ActivationKey::Destroy, @activation_key)
- respond :message => _("Deleted activation key '%s'") % params[:id], :status => 204
respond_for_destroy
end

api :GET, "/activation_keys/:id", N_("Show an activation key")

Makes it consistent with the other parts of the API:

root@li-lc-1017:/home/hoiroot# curl -K /opt/hoici/etc/curl-hoici.conf -H 'Content-Type: application/json' -d '{"per_page":9999}' -XDELETE https://localhost/katello/api/v2/activation_keys/28 {}
root@li-lc-1017:/home/hoiroot#

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual results:

Expected results:
Consistent responses throughout the API

Additional info:

Actions #1

Updated by The Foreman Bot over 9 years ago

  • Status changed from New to Ready For Testing
  • Target version set to 62
  • Pull request https://github.com/Katello/katello/pull/4854 added
  • Pull request deleted ()
Actions #2

Updated by Eric Helms over 9 years ago

  • translation missing: en.field_release set to 14
  • Triaged changed from No to Yes
Actions #3

Updated by Eric Helms over 9 years ago

  • Target version changed from 62 to 63
Actions #4

Updated by Eric Helms over 9 years ago

  • translation missing: en.field_release changed from 14 to 23
Actions #5

Updated by Eric Helms about 9 years ago

  • Target version changed from 63 to 66
Actions #6

Updated by Christine Fouant about 9 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF