Project

General

Profile

Actions

Bug #6674

closed

Can't create a host via the API because params aren't correctly wrapped

Added by David Davis over 9 years ago. Updated over 9 years ago.

Status:
Duplicate
Priority:
Normal
Assignee:
Category:
API
Target version:
-
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

When I try the following curl command, I get "Name can't be blank":

$ curl -k -u admin:changeme -H "Content-Type: application/json" -X POST http://localhost:3000/api/v2/hosts -d '{"name": "TEST"}'

It looks like the name param is getting wrapped in params[:base] instead of params[:host].


Related issues 2 (0 open2 closed)

Related to Foreman - Bug #5178: Users API requires parameters to be wrappedClosedJoseph Magen04/14/2014Actions
Is duplicate of Foreman - Bug #6696: API v2 - specify the key in which the parameters would be wrapped rather than passing model nameClosedJoseph Magen07/20/2014Actions
Actions #1

Updated by Thomas McKay over 9 years ago

  • Bugzilla link set to 1120800
Actions #2

Updated by Thomas McKay over 9 years ago

  • Assignee set to Thomas McKay
Actions #3

Updated by The Foreman Bot over 9 years ago

  • Status changed from New to Ready For Testing
  • Target version set to 1.8.0
  • Pull request https://github.com/theforeman/foreman/pull/1597 added
  • Pull request deleted ()
Actions #4

Updated by Daniel Lobato Garcia over 9 years ago

  • Status changed from Ready For Testing to Rejected

As explained in:

https://github.com/theforeman/foreman/pull/1597#issuecomment-49421858

API POST calls need to receive a wrapped object in a key with the name of the resource you want to create, it's the standard behavior in all POST calls in api/v2

http://theforeman.org/api_v2.html

Actions #5

Updated by David Davis over 9 years ago

  • Status changed from Rejected to Assigned

Please see the following bug:

http://projects.theforeman.org/issues/4181

The API accepts wrapped or unwrapped. The issue here is that when you send unwrapped params, it wraps it in the wrong root node (base instead of host).

Actions #6

Updated by The Foreman Bot over 9 years ago

  • Status changed from Assigned to Ready For Testing
Actions #7

Updated by Joseph Magen over 9 years ago

  • Status changed from Ready For Testing to Duplicate
Actions #8

Updated by Dominic Cleal over 9 years ago

  • Related to Bug #5178: Users API requires parameters to be wrapped added
Actions #9

Updated by Dominic Cleal over 9 years ago

  • Target version deleted (1.8.0)
Actions #10

Updated by Dominic Cleal over 9 years ago

  • Is duplicate of Bug #6696: API v2 - specify the key in which the parameters would be wrapped rather than passing model name added
Actions

Also available in: Atom PDF