Skip to content

CTOR-2330 - fix qwinsta parsing - #6300

Merged
omercier merged 2 commits into
developfrom
CTOR-2330
Sep 17, 2026
Merged

omercier merged 2 commits into
developfrom
CTOR-2330

Conversation

@s-duret

@s-duret s-duret commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Description

I fixed the column parsing to handle localized outputs (e.g., German).

Changes:
File: sessions.pm — replaced byte/position-based parsing with a more robust parser that splits on two-or-more spaces (with a whitespace fallback).

Why:
Byte/position parsing could fail with localized/multibyte headers (e.g., German characters).
The new parsing handles those cases better.

Fixes # CTOR-2330

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Functionality enhancement or optimization (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software

Checklist

  • I have followed the coding style guidelines provided by Centreon
  • I have commented my code, especially hard-to-understand areas of the PR.
  • I have rebased my development branch on the base branch (develop).

@s-duret
s-duret requested a review from a team as a code owner July 10, 2026 12:37
@s-duret
s-duret requested review from omercier and removed request for a team July 10, 2026 12:37
@s-duret s-duret changed the title fix qwinsta parsing CTOR-2330 - fix qwinsta parsing Jul 10, 2026
@s-duret
s-duret requested a review from a team as a code owner July 10, 2026 13:37
@s-duret
s-duret requested review from sdepassio and removed request for a team July 10, 2026 13:37
@omercier
omercier merged commit a5b1e54 into develop Sep 17, 2026
30 checks passed
@omercier
omercier deleted the CTOR-2330 branch September 17, 2026 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants