Project

General

Profile

Actions
Copy link

Bug #24703

closed

OS type / name are not always compared lower case

Added by François ARMAND 9 months ago. Updated 8 months ago.

Status:
Released
Priority:
N/A
Assignee:
Vincent MEMBRÉ
Category:
Web - Nodes & inventories
Target version:
8.1.1
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:
No

Description

Our parse OS method expects lower case comparison, but is not always called with lower case values.
Since the contract is not clear, it should toLowerCase by itself.

Currently, it cause problems when we are unserializing nodes from JSON facts.

Can only be corrected in 8.1 since unserialisation was added here.

Actions
Copy link
 #1

Updated by François ARMAND 9 months ago

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

Updated by François ARMAND 9 months ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/5595
Actions
Copy link
 #3

Updated by Vincent MEMBRÉ 8 months ago

  • Target version changed from 8.1.0 to 8.1.1
Actions
Copy link
 #4

Updated by Anonymous 8 months ago

  • Status changed from Pending technical review to Pending release
Actions
Copy link
 #5

Updated by François ARMAND 8 months ago

  • Fix check changed from To do to Checked

Inventory and node acceptation history still work wrt node OS

Actions
Copy link
 #6

Updated by Vincent MEMBRÉ 8 months ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 8.1.1 which was released today.

Actions
Copy link

Also available in: Atom PDF