Skip to content

CTOR-2330 - fix qwinsta parsing#6300

Open
s-duret wants to merge 2 commits into
developfrom
CTOR-2330
Open

CTOR-2330 - fix qwinsta parsing#6300
s-duret wants to merge 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
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.

2 participants