Project

General

Profile

Bug #17388

Bad JSON answer for api PUT /rules

Added by François ARMAND 5 months ago. Updated 2 months ago.

Status:
Released
Priority:
N/A
Category:
API
Target version:
Severity:
User visibility:
Effort required:
Priority:
0

Description

The return JSON is missing the array of rules, contrary to what is documented and what is done for directives/groups.
It is also displaying ruleCategories in place of rule:

Actual:

{
  "action": "createRule",
  "id": "e12b9720-66c0-4887-8026-f72cdc423a69",
  "result": "success",
  "data": {
    "ruleCategories": {
      "id": "e12b9720-66c0-4887-8026-f72cdc423a69",
      "displayName": "RTF generated rule",
      "shortDescription": "",
      "longDescription": "",
      "directives": [
        "b078d18e-7a99-4bd5-8386-43eaf4f3669f" 
      ],
      "targets": [
        "policyServer:root" 
      ],
      "enabled": true,
      "system": false,
      "tags": []
    }
  }
}

VS (should):

{
  "action": "createRule",
  "id": "e12b9720-66c0-4887-8026-f72cdc423a69",
  "result": "success",
  "data": {
    "rules": [ {
      "id": "e12b9720-66c0-4887-8026-f72cdc423a69",
      "displayName": "RTF generated rule",
      "shortDescription": "",
      "longDescription": "",
      "directives": [
        "b078d18e-7a99-4bd5-8386-43eaf4f3669f" 
      ],
      "targets": [
        "policyServer:root" 
      ],
      "enabled": true,
      "system": false,
      "tags": []
    }
    ]
  }
}


Subtasks

Bug #17455: API change in rules upmerge breaks rudder 6.1ReleasedVincent MEMBRÉActions
Bug #17456: Parent ticket on rule api need to be backported on 5.0ReleasedVincent MEMBRÉActions
#1

Updated by Vincent MEMBRÉ 5 months ago

  • Status changed from New to In progress
  • Assignee set to Vincent MEMBRÉ
#2

Updated by Vincent MEMBRÉ 5 months ago

  • Assignee changed from Vincent MEMBRÉ to François ARMAND
  • Pull Request set to https://github.com/Normation/rudder/pull/2969
#3

Updated by Vincent MEMBRÉ 4 months ago

  • Description updated (diff)

Corrected the expected json to

{
  "action": "createRule",
  "id": "e12b9720-66c0-4887-8026-f72cdc423a69",
  "result": "success",
  "data": {
    "rules": [ {
      "id": "e12b9720-66c0-4887-8026-f72cdc423a69",
      "displayName": "RTF generated rule",
      "shortDescription": "",
      "longDescription": "",
      "directives": [
        "b078d18e-7a99-4bd5-8386-43eaf4f3669f" 
      ],
      "targets": [
        "policyServer:root" 
      ],
      "enabled": true,
      "system": false,
      "tags": []
    }
    ]
  }
}

#4

Updated by Vincent MEMBRÉ 4 months ago

  • Status changed from In progress to Pending release
#6

Updated by Vincent MEMBRÉ 2 months ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 5.0.18 and 6.1.2 which were released today.

Also available in: Atom PDF