Project

General

Profile

Actions

Bug #3267

closed

Foreman 1.3 breaks API v1 compatibility for /api/hosts

Added by Daniel Lobato Garcia over 10 years ago. Updated almost 6 years ago.

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

Description

1.2 v1 hosts API used to have a nested hash for showing the environment, such as:

{"host" : { "name" : "foreman.redhat.com", "environment":{"environment":{"id":16,"name":"production"}}}

However in version 1.3, the returned JSON is slightly different (misses a nested hash for environments)

{"host" : { "name" : "foreman.redhat.com", "environment" : {"id":16,"name":"production"}}}

I think the latter looks better and v2 should be like that. Nonetheless this change might break retrocompatibility for some apps using v1 API.

I believe the error happens since this commit: https://github.com/theforeman/foreman/commit/76e5dd41bd575f7e6df7c7422660510216a9b964#diff-e9c3a7122d98b9c9f497817c8ca87843

Actions

Also available in: Atom PDF