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 over 4 years ago. Updated 21 days ago.

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

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 over 4 years ago

  • Target version changed from 2.10.4 to 2.10.5

#2 Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 2.10.5 to 2.10.6

#3 Updated by Matthieu CERDA about 4 years ago

  • Target version changed from 2.10.6 to 2.10.7

#4 Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 2.10.7 to 2.10.8

#5 Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 2.10.8 to 2.10.9

#6 Updated by Vincent MEMBRÉ almost 4 years ago

  • Target version changed from 2.10.9 to 2.10.10

#7 Updated by Vincent MEMBRÉ almost 4 years ago

  • Target version changed from 2.10.10 to 2.10.11

#8 Updated by Vincent MEMBRÉ almost 4 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É over 3 years ago

  • Target version changed from 2.10.15 to 2.10.16

#13 Updated by Vincent MEMBRÉ over 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É about 3 years ago

  • Target version changed from 2.10.18 to 2.10.19

#16 Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 2.10.19 to 2.10.20

#17 Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 2.10.20 to 277

#18 Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 277 to 2.11.18

#19 Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 2.11.18 to 2.11.19

#20 Updated by Vincent MEMBRÉ almost 3 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É over 2 years ago

  • Target version changed from 2.11.23 to 2.11.24

#25 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.11.24 to 308

#26 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 308 to 3.1.14

#27 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 3.1.14 to 3.1.15

#28 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 3.1.15 to 3.1.16

#29 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 3.1.16 to 3.1.17

#30 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 3.1.17 to 3.1.18

#31 Updated by Vincent MEMBRÉ almost 2 years 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 over 1 year ago

  • Priority changed from 14 to 27

#39 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.22 to 3.1.23

#40 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.23 to 3.1.24

#41 Updated by Vincent MEMBRÉ about 1 year ago

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

#42 Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 3.1.25 to 387

#43 Updated by Vincent MEMBRÉ 12 months ago

  • Target version changed from 387 to 4.1.10

#44 Updated by Vincent MEMBRÉ 10 months ago

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

#45 Updated by Vincent MEMBRÉ 8 months ago

  • Target version changed from 4.1.11 to 4.1.12

#46 Updated by Vincent MEMBRÉ 7 months ago

  • Target version changed from 4.1.12 to 4.1.13

#47 Updated by Vincent MEMBRÉ 5 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 4 months ago

  • Target version changed from 4.1.14 to 4.1.15

#49 Updated by Vincent MEMBRÉ 2 months ago

  • Target version changed from 4.1.15 to 4.1.16

#50 Updated by Vincent MEMBRÉ about 1 month ago

  • Target version changed from 4.1.16 to 4.1.17
  • Priority changed from 30 to 31

#51 Updated by Vincent MEMBRÉ 21 days ago

  • Target version changed from 4.1.17 to 4.1.18
  • Priority changed from 31 to 0

Also available in: Atom PDF