Project

General

Profile

Actions

Bug #23332

closed

Serde errors do not point to the correct line number

Added by Alexis Mousset about 1 year ago. Updated 11 months ago.

Status:
Released
Priority:
N/A
Category:
rudderc
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
Checked
Regression:
No

Description

Due to the way items are deserialized.

The other problem is that higher level diagnostics can't refer to a source line.

No simple fix:

Leads:

We can start with a simple solution, which it to deserialize into a simpler structure to handle basic stuff, and then convert to the actual technique and manually handle errors.

Actions #1

Updated by Alexis Mousset about 1 year ago

  • Status changed from New to In progress
  • Assignee set to Alexis Mousset
Actions #2

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 8.0.0~beta1 to 8.0.0~beta2
Actions #3

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 8.0.0~beta2 to 8.0.0~beta3
Actions #4

Updated by Alexis Mousset about 1 year ago

  • Target version changed from 8.0.0~beta3 to 8.1.0~alpha1
Actions #5

Updated by Alexis Mousset 12 months ago

  • Description updated (diff)
Actions #6

Updated by Alexis Mousset 12 months ago

  • Description updated (diff)
  • Target version changed from 8.1.0~alpha1 to 8.0.4
Actions #7

Updated by Alexis Mousset 12 months ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Alexis Mousset to Félix DALLIDET
  • Pull Request set to https://github.com/Normation/rudder/pull/5226
Actions #8

Updated by Alexis Mousset 12 months ago

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

Updated by Alexis Mousset 11 months ago

  • Fix check changed from To do to Checked
Actions #10

Updated by Vincent MEMBRÉ 11 months ago

  • Status changed from Pending release to Released

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

Actions

Also available in: Atom PDF