Project

General

Profile

Actions

Bug #3881

closed

API : URL ending with / are seen like /[empty string parameter]

Added by Vincent MEMBRÉ over 9 years ago. Updated 8 months ago.

Status:
Released
Priority:
N/A
Category:
Documentation
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Medium
Priority:
0
Regression:

Description

When using the API, if the URL ends with a / it does not call the correct function, it tries instead to call the next function with an empty value:

calling /api/rules/ should return list of all rule, but instead it looks for the rule details of Rule with an empty id ("")

We should handle those URL correctly

Actions #1

Updated by Nicolas PERRON about 9 years ago

  • Target version changed from 2.7.1 to 2.7.2
Actions #2

Updated by Nicolas PERRON about 9 years ago

  • Target version changed from 2.7.2 to 2.7.3
Actions #3

Updated by Nicolas PERRON about 9 years ago

  • Target version changed from 2.7.3 to 2.7.4
Actions #4

Updated by Nicolas PERRON about 9 years ago

  • Target version changed from 2.7.4 to 2.7.5
Actions #5

Updated by Nicolas PERRON about 9 years ago

  • Target version changed from 2.7.5 to 2.7.6
Actions #6

Updated by Nicolas PERRON about 9 years ago

  • Target version changed from 2.7.6 to 2.7.7
Actions #7

Updated by Vincent MEMBRÉ almost 9 years ago

  • Target version changed from 2.7.7 to 2.7.8
Actions #8

Updated by Vincent MEMBRÉ almost 9 years ago

  • Target version changed from 2.7.8 to 2.8.3

Since 2.7.* is not supported anymore, targeting in 2.8

Actions #9

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.8.3 to 2.8.4
Actions #10

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.8.4 to 133
Actions #11

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 133 to 2.9.5
Actions #12

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.9.5 to 2.9.6
Actions #13

Updated by Jonathan CLARKE over 8 years ago

  • Target version changed from 2.9.6 to 2.9.7
Actions #14

Updated by Jonathan CLARKE over 8 years ago

  • Target version changed from 2.9.7 to 150
Actions #15

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 150 to 2.10.4
Actions #16

Updated by Nicolas PERRON over 8 years ago

  • Target version changed from 2.10.4 to 2.10.5
Actions #17

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.10.5 to 2.10.6
Actions #18

Updated by Matthieu CERDA about 8 years ago

  • Target version changed from 2.10.6 to 2.10.7
Actions #19

Updated by Vincent MEMBRÉ about 8 years ago

  • Target version changed from 2.10.7 to 2.10.8
Actions #20

Updated by Vincent MEMBRÉ almost 8 years ago

  • Target version changed from 2.10.8 to 2.10.9
Actions #21

Updated by Vincent MEMBRÉ almost 8 years ago

  • Target version changed from 2.10.9 to 2.10.10
Actions #22

Updated by Vincent MEMBRÉ almost 8 years ago

  • Target version changed from 2.10.10 to 2.10.11
Actions #23

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 2.10.11 to 2.10.12
Actions #24

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 2.10.12 to 2.10.13
Actions #25

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 2.10.13 to 2.10.14
Actions #26

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 2.10.14 to 2.10.15
Actions #27

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 2.10.15 to 2.10.16
Actions #28

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 2.10.16 to 2.10.17
Actions #29

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 2.10.17 to 2.10.18
Actions #30

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 2.10.18 to 2.10.19
Actions #31

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 2.10.19 to 2.10.20
Actions #32

Updated by Vincent MEMBRÉ almost 7 years ago

  • Target version changed from 2.10.20 to 2.11.18
Actions #33

Updated by Vincent MEMBRÉ almost 7 years ago

  • Target version changed from 2.11.18 to 2.11.19
Actions #34

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 2.11.19 to 2.11.20
Actions #35

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 2.11.20 to 2.11.21
Actions #36

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 2.11.21 to 2.11.22
Actions #37

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 2.11.22 to 2.11.23
Actions #38

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 2.11.23 to 2.11.24
Actions #39

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 2.11.24 to 308
Actions #40

Updated by Vincent MEMBRÉ about 6 years ago

  • Target version changed from 308 to 3.1.14
Actions #41

Updated by Vincent MEMBRÉ about 6 years ago

  • Target version changed from 3.1.14 to 3.1.15
Actions #42

Updated by Vincent MEMBRÉ about 6 years ago

  • Target version changed from 3.1.15 to 3.1.16
Actions #43

Updated by Vincent MEMBRÉ about 6 years ago

  • Target version changed from 3.1.16 to 3.1.17
Actions #44

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 3.1.17 to 3.1.18
Actions #45

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 3.1.18 to 3.1.19
Actions #46

Updated by Jonathan CLARKE over 5 years ago

  • Severity set to Minor - inconvenience | misleading | easy workaround
Actions #47

Updated by François ARMAND over 5 years ago

  • User visibility set to Operational - other Techniques | Technique editor | Rudder settings

Strictly speaking, it is not really a bug and changing the behavior may lead to cases that are no longer supported (the one with the empty value). So I'm not sure we should correct that - even if I'm the first to curse when I'm mistaking /api/foo/ for /api/foo

Actions #48

Updated by François ARMAND over 5 years ago

  • Effort required set to Medium
Actions #49

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 3.1.19 to 3.1.20
  • Priority set to 0
Actions #50

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 3.1.20 to 3.1.21
Actions #51

Updated by François ARMAND over 5 years ago

  • Priority changed from 0 to 14
Actions #52

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 3.1.21 to 3.1.22
Actions #53

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 3.1.22 to 3.1.23
Actions #54

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 3.1.23 to 3.1.24
Actions #55

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 3.1.24 to 3.1.25
  • Priority changed from 14 to 15
Actions #56

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 3.1.25 to 387
Actions #57

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 387 to 4.1.10
Actions #58

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 4.1.10 to 4.1.11
Actions #59

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 4.1.11 to 4.1.12
Actions #60

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 4.1.12 to 4.1.13
Actions #61

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 4.1.13 to 4.1.14
  • Priority changed from 15 to 16
Actions #62

Updated by Benoît PECCATTE over 4 years ago

  • Target version changed from 4.1.14 to 4.1.15
Actions #63

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 4.1.15 to 4.1.16
Actions #64

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 4.1.16 to 4.1.17
Actions #65

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 4.1.17 to 4.1.18
  • Priority changed from 16 to 0
Actions #66

Updated by Vincent MEMBRÉ almost 4 years ago

  • Target version changed from 4.1.18 to 4.1.19
Actions #67

Updated by Alexis Mousset almost 4 years ago

  • Target version changed from 4.1.19 to 4.1.20
Actions #68

Updated by Alexis Mousset over 3 years ago

  • Target version changed from 4.1.20 to 588
Actions #69

Updated by Alexis Mousset over 3 years ago

  • Assignee deleted (Vincent MEMBRÉ)
  • Target version changed from 588 to 5.0.13
Actions #70

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 5.0.13 to 5.0.14
Actions #71

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 5.0.14 to 5.0.15
Actions #72

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 5.0.15 to 5.0.16
Actions #73

Updated by Alexis Mousset almost 3 years ago

  • Target version changed from 5.0.16 to 5.0.17
Actions #74

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 5.0.17 to 5.0.18
Actions #75

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 5.0.18 to 5.0.19
Actions #76

Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 5.0.19 to 5.0.20
Actions #77

Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 5.0.20 to 797
Actions #78

Updated by Benoît PECCATTE over 1 year ago

  • Target version changed from 797 to 6.1.14
Actions #79

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 6.1.14 to 6.1.15
Actions #80

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 6.1.15 to 6.1.16
Actions #81

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 6.1.16 to 6.1.17
Actions #82

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 6.1.17 to 6.1.18
Actions #83

Updated by Vincent MEMBRÉ 12 months ago

  • Target version changed from 6.1.18 to 6.1.19
Actions #84

Updated by François ARMAND 10 months ago

  • Subject changed from API : URL ending with / are not correctly handled to API : URL ending with / are seen like /[empty string parameter]

This is a known limitation that will not be corected without a major change => documenting it

Actions #85

Updated by François ARMAND 10 months ago

  • Category changed from API to Documentation
  • Target version changed from 6.1.19 to 6.2.13
Actions #86

Updated by François ARMAND 10 months ago

  • Status changed from New to In progress
  • Assignee set to François ARMAND
Actions #87

Updated by François ARMAND 10 months ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Alexis Mousset
  • Pull Request set to https://github.com/Normation/rudder/pull/4140
Actions #88

Updated by François ARMAND 10 months ago

  • Status changed from Pending technical review to Pending release
Actions #90

Updated by Vincent MEMBRÉ 8 months ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.2.13, 7.0.2 and 7.1.0~rc1 which were released today.

Actions

Also available in: Atom PDF