Project

General

Profile

Actions

Bug #6647

closed

Foreman organization API destroy routes (DELETE /api/v2/organizations/1 and DELETE /api/v1/organizations/1) are still accessible

Added by Eric Helms over 9 years ago. Updated over 5 years ago.

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

Related issues 3 (0 open3 closed)

Related to Katello - Bug #6707: Remove or update the V1 APIRejected07/21/2014Actions
Related to Katello - Bug #6708: Have the /api/v2/organization routes point to our Organization controller in KatelloRejected07/21/2014Actions
Related to Foreman - Feature #9375: Find a way for plugins to easily override the behavior of taxonomies controller actionsRejected02/13/2015Actions
Actions #1

Updated by Partha Aji over 9 years ago

  • Bugzilla link set to 1120335
Actions #2

Updated by David Davis over 9 years ago

  • Priority changed from Normal to Low
Actions #3

Updated by The Foreman Bot over 9 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/4441 added
  • Pull request deleted ()
Actions #4

Updated by David Davis over 9 years ago

  • Status changed from Ready For Testing to New
  • Assignee deleted (David Davis)
  • Pull request added
  • Pull request deleted (https://github.com/Katello/katello/pull/4441)
Actions #5

Updated by David Davis over 9 years ago

  • Subject changed from Organization destroy API (/api/v2/organizations/1) does not use dynflow destroy to Foreman organization API (e.g. /api/v2/organizations/1) is still accessible

I was trying to approach this by pointing the route at the katello org controller. However, I think the better solution though is to remove the old routes. I don't think there should exist routes that aren't in our api docs.

Actions #6

Updated by The Foreman Bot over 9 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/4463 added
  • Pull request deleted ()
Actions #7

Updated by David Davis over 9 years ago

  • Assignee set to David Davis
Actions #8

Updated by David Davis over 9 years ago

  • Related to Bug #6707: Remove or update the V1 API added
Actions #9

Updated by David Davis over 9 years ago

  • Related to Bug #6708: Have the /api/v2/organization routes point to our Organization controller in Katello added
Actions #10

Updated by David Davis over 9 years ago

  • Subject changed from Foreman organization API (e.g. /api/v2/organizations/1) is still accessible to Foreman organization API destroy routes (DELETE /api/v2/organizations/1 and DELETE /api/v1/organizations/1) are still accessible

Making this more specific to org destroy since I need to also fix v1. Plus, #6708 should fix the Foreman organizations API.

Actions #11

Updated by David Davis over 9 years ago

  • Priority changed from Low to Normal
Actions #12

Updated by David Davis over 9 years ago

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

Updated by Eric Helms over 9 years ago

  • translation missing: en.field_release set to 13
Actions #14

Updated by David Davis about 9 years ago

  • Related to Feature #9375: Find a way for plugins to easily override the behavior of taxonomies controller actions added
Actions

Also available in: Atom PDF