Project

General

Profile

Actions
Copy link

Bug #17388

closed

Bad JSON answer for api PUT /rules

Bug #17388: Bad JSON answer for api PUT /rules

Added by François ARMAND about 6 years ago. Updated almost 6 years ago.

Status:
Released
Priority:
N/A
Assignee:
François ARMAND
Category:
API
Target version:
5.0.18
Pull Request:
https://github.com/Normation/rudder/p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
Checked
Regression:

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 2 (0 open — 2 closed)

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

Updated by Vincent MEMBRÉ about 6 years ago Actions
Copy link
 #1

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

Updated by Vincent MEMBRÉ about 6 years ago Actions
Copy link
 #2

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

Updated by Vincent MEMBRÉ about 6 years ago Actions
Copy link
 #3

  • 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": []
    }
    ]
  }
}

Updated by Vincent MEMBRÉ about 6 years ago Actions
Copy link
 #4

  • Status changed from In progress to Pending release

Updated by François ARMAND almost 6 years ago Actions
Copy link
 #5

  • Fix check changed from To do to Checked

Updated by Vincent MEMBRÉ almost 6 years ago Actions
Copy link
 #6

  • 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.

Actions
Copy link

Also available in: PDF Atom