Bug #3881: API : URL ending with / are seen like /[empty string parameter] - Rudder - Issue Tracker

Actions

Bug #3881

closed

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

Added by Vincent MEMBRÉ about 11 years ago. Updated over 2 years ago.

Status:

Released

Priority:

N/A

Assignee:

Category:

Documentation

Target version:

Pull Request:

https://github.com/Normation/rudder/p...

Severity:

Minor - inconvenience | misleading | easy workaround

UX impact:

User visibility:

Operational - other Techniques | Technique editor | Rudder settings

Effort required:

Medium

Priority:

0

Name check:

Fix check:

Checked

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 11 years ago

Target version changed from 2.7.1 to 2.7.2

Actions

#2

Updated by Nicolas PERRON about 11 years ago

Target version changed from 2.7.2 to 2.7.3

Actions

#3

Updated by Nicolas PERRON about 11 years ago

Target version changed from 2.7.3 to 2.7.4

Actions

#4

Updated by Nicolas PERRON about 11 years ago

Target version changed from 2.7.4 to 2.7.5

Actions

#5

Updated by Nicolas PERRON about 11 years ago

Target version changed from 2.7.5 to 2.7.6

Actions

#6

Updated by Nicolas PERRON about 11 years ago

Target version changed from 2.7.6 to 2.7.7

Actions

#7

Updated by Vincent MEMBRÉ almost 11 years ago

Target version changed from 2.7.7 to 2.7.8

Actions

#8

Updated by Vincent MEMBRÉ almost 11 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 10 years ago

Target version changed from 2.8.3 to 2.8.4

Actions

#10

Updated by Vincent MEMBRÉ over 10 years ago

Target version changed from 2.8.4 to 133

Actions

#11

Updated by Vincent MEMBRÉ over 10 years ago

Target version changed from 133 to 2.9.5

Actions

#12

Updated by Vincent MEMBRÉ over 10 years ago

Target version changed from 2.9.5 to 2.9.6

Actions

#13

Updated by Jonathan CLARKE over 10 years ago

Target version changed from 2.9.6 to 2.9.7

Actions

#14

Updated by Jonathan CLARKE over 10 years ago

Target version changed from 2.9.7 to 150

Actions

#15

Updated by Vincent MEMBRÉ over 10 years ago

Target version changed from 150 to 2.10.4

Actions

#16

Updated by Nicolas PERRON over 10 years ago

Target version changed from 2.10.4 to 2.10.5

Actions

#17

Updated by Vincent MEMBRÉ about 10 years ago

Target version changed from 2.10.5 to 2.10.6

Actions

#18

Updated by Matthieu CERDA about 10 years ago

Target version changed from 2.10.6 to 2.10.7

Actions

#19

Updated by Vincent MEMBRÉ about 10 years ago

Target version changed from 2.10.7 to 2.10.8

Actions

#20

Updated by Vincent MEMBRÉ almost 10 years ago

Target version changed from 2.10.8 to 2.10.9

Actions

#21

Updated by Vincent MEMBRÉ almost 10 years ago

Target version changed from 2.10.9 to 2.10.10

Actions

#22

Updated by Vincent MEMBRÉ almost 10 years ago

Target version changed from 2.10.10 to 2.10.11

Actions

#23

Updated by Vincent MEMBRÉ over 9 years ago

Target version changed from 2.10.11 to 2.10.12

Actions

#24

Updated by Vincent MEMBRÉ over 9 years ago

Target version changed from 2.10.12 to 2.10.13

Actions

#25

Updated by Vincent MEMBRÉ over 9 years ago

Target version changed from 2.10.13 to 2.10.14

Actions

#26

Updated by Vincent MEMBRÉ over 9 years ago

Target version changed from 2.10.14 to 2.10.15

Actions

#27

Updated by Vincent MEMBRÉ over 9 years ago

Target version changed from 2.10.15 to 2.10.16

Actions

#28

Updated by Vincent MEMBRÉ over 9 years ago

Target version changed from 2.10.16 to 2.10.17

Actions

#29

Updated by Vincent MEMBRÉ about 9 years ago

Target version changed from 2.10.17 to 2.10.18

Actions

#30

Updated by Vincent MEMBRÉ about 9 years ago

Target version changed from 2.10.18 to 2.10.19

Actions

#31

Updated by Vincent MEMBRÉ about 9 years ago

Target version changed from 2.10.19 to 2.10.20

Actions

#32

Updated by Vincent MEMBRÉ almost 9 years ago

Target version changed from 2.10.20 to 2.11.18

Actions

#33

Updated by Vincent MEMBRÉ almost 9 years ago

Target version changed from 2.11.18 to 2.11.19

Actions

#34

Updated by Vincent MEMBRÉ over 8 years ago

Target version changed from 2.11.19 to 2.11.20

Actions

#35

Updated by Vincent MEMBRÉ over 8 years ago

Target version changed from 2.11.20 to 2.11.21

Actions

#36

Updated by Vincent MEMBRÉ over 8 years ago

Target version changed from 2.11.21 to 2.11.22

Actions

#37

Updated by Vincent MEMBRÉ over 8 years ago

Target version changed from 2.11.22 to 2.11.23

Actions

#38

Updated by Vincent MEMBRÉ over 8 years ago

Target version changed from 2.11.23 to 2.11.24

Actions

#39

Updated by Vincent MEMBRÉ about 8 years ago

Target version changed from 2.11.24 to 308

Actions

#40

Updated by Vincent MEMBRÉ about 8 years ago

Target version changed from 308 to 3.1.14

Actions

#41

Updated by Vincent MEMBRÉ about 8 years ago

Target version changed from 3.1.14 to 3.1.15

Actions

#42

Updated by Vincent MEMBRÉ about 8 years ago

Target version changed from 3.1.15 to 3.1.16

Actions

#43

Updated by Vincent MEMBRÉ about 8 years ago

Target version changed from 3.1.16 to 3.1.17

Actions

#44

Updated by Vincent MEMBRÉ almost 8 years ago

Target version changed from 3.1.17 to 3.1.18

Actions

#45

Updated by Vincent MEMBRÉ almost 8 years ago

Target version changed from 3.1.18 to 3.1.19

Actions

#46

Updated by Jonathan CLARKE over 7 years ago

Severity set to Minor - inconvenience | misleading | easy workaround

Actions

#47

Updated by François ARMAND over 7 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 7 years ago

Effort required set to Medium

Actions

#49

Updated by Vincent MEMBRÉ over 7 years ago

Target version changed from 3.1.19 to 3.1.20
Priority set to 0

Actions

#50

Updated by Vincent MEMBRÉ over 7 years ago

Target version changed from 3.1.20 to 3.1.21

Actions

#51

Updated by François ARMAND over 7 years ago

Priority changed from 0 to 14

Actions

#52

Updated by Vincent MEMBRÉ over 7 years ago

Target version changed from 3.1.21 to 3.1.22

Actions

#53

Updated by Vincent MEMBRÉ over 7 years ago

Target version changed from 3.1.22 to 3.1.23

Actions

#54

Updated by Vincent MEMBRÉ about 7 years ago

Target version changed from 3.1.23 to 3.1.24

Actions

#55

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

Target version changed from 3.1.25 to 387

Actions

#57

Updated by Vincent MEMBRÉ almost 7 years ago

Target version changed from 387 to 4.1.10

Actions

#58

Updated by Vincent MEMBRÉ almost 7 years ago

Target version changed from 4.1.10 to 4.1.11

Actions

#59

Updated by Vincent MEMBRÉ over 6 years ago

Target version changed from 4.1.11 to 4.1.12

Actions

#60

Updated by Vincent MEMBRÉ over 6 years ago

Target version changed from 4.1.12 to 4.1.13

Actions

#61

Updated by Vincent MEMBRÉ over 6 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 6 years ago

Target version changed from 4.1.14 to 4.1.15

Actions

#63

Updated by Vincent MEMBRÉ about 6 years ago

Target version changed from 4.1.15 to 4.1.16

Actions

#64

Updated by Vincent MEMBRÉ about 6 years ago

Target version changed from 4.1.16 to 4.1.17

Actions

#65

Updated by Vincent MEMBRÉ almost 6 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 6 years ago

Target version changed from 4.1.18 to 4.1.19

Actions

#67

Updated by Alexis Mousset almost 6 years ago

Target version changed from 4.1.19 to 4.1.20

Actions

#68

Updated by Alexis Mousset over 5 years ago

Target version changed from 4.1.20 to 588

Actions

#69

Updated by Alexis Mousset over 5 years ago

Assignee deleted (~~Vincent MEMBRÉ~~)
Target version changed from 588 to 5.0.13

Actions

#70

Updated by Vincent MEMBRÉ about 5 years ago

Target version changed from 5.0.13 to 5.0.14

Actions

#71

Updated by Vincent MEMBRÉ about 5 years ago

Target version changed from 5.0.14 to 5.0.15

Actions

#72

Updated by Vincent MEMBRÉ almost 5 years ago

Target version changed from 5.0.15 to 5.0.16

Actions

#73

Updated by Alexis Mousset almost 5 years ago

Target version changed from 5.0.16 to 5.0.17

Actions

#74

Updated by Vincent MEMBRÉ over 4 years ago

Target version changed from 5.0.17 to 5.0.18

Actions

#75

Updated by Vincent MEMBRÉ over 4 years ago

Target version changed from 5.0.18 to 5.0.19

Actions

#76

Updated by Vincent MEMBRÉ about 4 years ago

Target version changed from 5.0.19 to 5.0.20

Actions

#77

Updated by Vincent MEMBRÉ about 4 years ago

Target version changed from 5.0.20 to 797

Actions

#78

Updated by Benoît PECCATTE over 3 years ago

Target version changed from 797 to 6.1.14

Actions

#79

Updated by Vincent MEMBRÉ over 3 years ago

Target version changed from 6.1.14 to 6.1.15

Actions

#80

Updated by Vincent MEMBRÉ over 3 years ago

Target version changed from 6.1.15 to 6.1.16

Actions

#81

Updated by Vincent MEMBRÉ about 3 years ago

Target version changed from 6.1.16 to 6.1.17

Actions

#82

Updated by Vincent MEMBRÉ about 3 years ago

Target version changed from 6.1.17 to 6.1.18

Actions

#83

Updated by Vincent MEMBRÉ almost 3 years ago

Target version changed from 6.1.18 to 6.1.19

Actions

#84

Updated by François ARMAND almost 3 years 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 almost 3 years 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 almost 3 years ago

Status changed from New to In progress
Assignee set to François ARMAND

Actions

#87

Updated by François ARMAND almost 3 years 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

PR https://github.com/Normation/rudder/pull/4140

Actions

#88

Updated by François ARMAND almost 3 years ago

Status changed from Pending technical review to Pending release

Applied in changeset rudder|70e0f43f7560a53586d41a00900a7fcee4333e52.

Actions

#89

Updated by Alexis Mousset almost 3 years ago

Fix check set to Checked

Actions

#90

Updated by Vincent MEMBRÉ over 2 years 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