Project

General

Profile

Bug #5345

Modifying id of a parameter with API does not work but return a succes an launch a promise generation

Added by Nicolas PERRON about 4 years ago. Updated about 2 months ago.

Status:
New
Priority:
3
Assignee:
-
Category:
API
Target version:
Target version (plugin):
Severity:
Minor - inconvenience | misleading | easy workaround
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Pull Request:
Priority:
30

Description

After trying to rename a parameter from "rudder_file_edit_footer" to "my_param" with API the result is:

$ curl -k -H "X-API-Token: PojCJDhajkGqVx1TTmiZKNFZORKOYc8X" -X POST https://localhost/rudder/api/2/parameters/rudder_file_edit_footer?prettify=true -d "id=my_param" 

{
  "action": "updateParameter",
  "id": "rudder_file_edit_footer",
  "result": "success",
  "data": {
    "parameters": [
      {
        "id": "rudder_file_edit_footer",
        "value": "### Edited by Rudder ###",
        "description": "Default inform message put in footer of managed files by Rudder",
        "overridable": false
      }
    ]
  }
}

The id has not changed.

If we try to do it manually by the WebUI, this is not possible since the id textbox is not editable. On the Eventlogs, policy generations are logged but not any modification.

I suppose this action with API should return an error, not a success.

History

#1 Updated by Nicolas PERRON about 4 years ago

  • Target version changed from 2.10.4 to 2.10.5

#2 Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 2.10.5 to 2.10.6

#3 Updated by Matthieu CERDA almost 4 years ago

  • Target version changed from 2.10.6 to 2.10.7

#4 Updated by Vincent MEMBRÉ almost 4 years ago

  • Target version changed from 2.10.7 to 2.10.8

#5 Updated by Vincent MEMBRÉ almost 4 years ago

  • Target version changed from 2.10.8 to 2.10.9

#6 Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 2.10.9 to 2.10.10

#7 Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 2.10.10 to 2.10.11

#8 Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 2.10.11 to 2.10.12

#9 Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 2.10.12 to 2.10.13

#10 Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 2.10.13 to 2.10.14

#11 Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 2.10.14 to 2.10.15

#12 Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 2.10.15 to 2.10.16

#13 Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 2.10.16 to 2.10.17

#14 Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 2.10.17 to 2.10.18

#15 Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 2.10.18 to 2.10.19

#16 Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 2.10.19 to 2.10.20

#17 Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 2.10.20 to 277

#18 Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 277 to 2.11.18

#19 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.11.18 to 2.11.19

#20 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.11.19 to 2.11.20

#21 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.11.20 to 2.11.21

#22 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.11.21 to 2.11.22

#23 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.11.22 to 2.11.23

#24 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 2.11.23 to 2.11.24

#25 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 2.11.24 to 308

#26 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 308 to 3.1.14

#27 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 3.1.14 to 3.1.15

#28 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 3.1.15 to 3.1.16

#29 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 3.1.16 to 3.1.17

#30 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 3.1.17 to 3.1.18

#31 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.18 to 3.1.19

#32 Updated by François ARMAND over 1 year ago

  • Severity set to Minor - inconvenience | misleading | easy workaround
  • User visibility set to Operational - other Techniques | Technique editor | Rudder settings
  • Priority set to 0

#33 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.19 to 3.1.20

#34 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.20 to 3.1.21

#35 Updated by François ARMAND over 1 year ago

The behavior is the expected one: id can not be modified. So we should just return an error here.

#36 Updated by François ARMAND over 1 year ago

  • Priority changed from 0 to 14

#37 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.21 to 3.1.22

#38 Updated by Benoît PECCATTE about 1 year ago

  • Priority changed from 14 to 27

#39 Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 3.1.22 to 3.1.23

#40 Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 3.1.23 to 3.1.24

#41 Updated by Vincent MEMBRÉ 11 months ago

  • Target version changed from 3.1.24 to 3.1.25
  • Priority changed from 27 to 28

#42 Updated by Vincent MEMBRÉ 10 months ago

  • Target version changed from 3.1.25 to 387

#43 Updated by Vincent MEMBRÉ 9 months ago

  • Target version changed from 387 to 4.1.10

#44 Updated by Vincent MEMBRÉ 7 months ago

  • Target version changed from 4.1.10 to 4.1.11
  • Priority changed from 28 to 29

#45 Updated by Vincent MEMBRÉ 5 months ago

  • Target version changed from 4.1.11 to 4.1.12

#46 Updated by Vincent MEMBRÉ 4 months ago

  • Target version changed from 4.1.12 to 4.1.13

#47 Updated by Vincent MEMBRÉ 2 months ago

  • Target version changed from 4.1.13 to 4.1.14
  • Priority changed from 29 to 30

#48 Updated by Benoît PECCATTE about 2 months ago

  • Target version changed from 4.1.14 to 4.1.15

Also available in: Atom PDF