Project

General

Profile

Actions
Copy link

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 10 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
David Davis
Category:
API
Target version:
Katello 2.0
Difficulty:
medium
Triaged:
Yes
Bugzilla link:
1120335
Pull request:
https://github.com/Katello/katello/pull/4463
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Related issues 3 (0 open — 3 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
Copy link
 #1

Updated by Partha Aji over 10 years ago

  • Bugzilla link set to 1120335
Actions
Copy link
 #2

Updated by David Davis over 10 years ago

  • Priority changed from Normal to Low
Actions
Copy link
 #3

Updated by The Foreman Bot over 10 years ago

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

Updated by David Davis over 10 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
Copy link
 #5

Updated by David Davis over 10 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
Copy link
 #6

Updated by The Foreman Bot over 10 years ago

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

Updated by David Davis over 10 years ago

  • Assignee set to David Davis
Actions
Copy link
 #8

Updated by David Davis over 10 years ago

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

Updated by David Davis over 10 years ago

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

Updated by David Davis over 10 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
Copy link
 #11

Updated by David Davis over 10 years ago

  • Priority changed from Low to Normal
Actions
Copy link
 #12

Updated by David Davis over 10 years ago

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

Updated by Eric Helms over 10 years ago

  • Translation missing: en.field_release set to 13
Actions
Copy link
 #14

Updated by David Davis almost 10 years ago

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

Also available in: Atom PDF