Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

WQL array properties: add the trailing separator to match wmi-java

Fechada
  • Algo errado?
#201 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

@bertysentry já está trabalhando nisso.

Desde 9/10/2026.

  • #204 de @bertysentry — aberto

Avaliação

Dificuldade
3/5
Tempo estimado
1-2 dias
Facilidade para iniciantes
58/100
Tipo de issue
Funcionalidade
Clareza
Razoavelmente clara
Status de atividade
Ativa
Stack de tecnologia
java
Domínio
backend

Direção de pesquisa

Start by reading the WqlRow Javadoc and the array-property rendering, then check how WinRMClient.Builder.arraySeparator(String) configures the separator. Run the project tests and check the array-rendering cases. Done means array elements have the agreed trailing-separator behavior, the empty-array choice is explicit, and WqlRow Javadoc, wql.md and the CHANGELOG describe it.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

enhancement

Problem

Since 3.0.00 (#189), a WMI array property is rendered by joining its elements with the separator: 192.0.2.10|fe80::1.

wmi-java renders the same array with a trailing separator: WmiStringConverter joins with Collectors.joining(arraySeparator, "", arraySeparator), after stripping the separator from each element, which yields 192.0.2.10|fe80::1| (an empty array yields |).

MetricsHub connectors run the same WQL queries through both libraries (WMI extension on Windows collectors, WinRM extension elsewhere), and their AWK scripts and split computes were written against the wmi-java output. The two libraries must produce the same string.

Proposal

Align winrm-java on wmi-java:

  • append the separator after the last element (a|b|), including for a single element (a|);
  • strip the separator from each element before joining, as wmi-java does;
  • keep | as the default and WinRMClient.Builder.arraySeparator(String) as the override.

Decide explicitly what an empty array renders as (wmi-java gives |; an empty string may be more useful) and document it in the WqlRow Javadoc, wql.md and the CHANGELOG.

The change only affects array properties, which 2.x truncated to their last element anyway, so no caller can depend on the 3.0.00 form yet.

Linguagem predominante
Java
Estrelas
15
Forks
4
Merge médio
11h 58min
PRs com merge (30d)
20

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de MetricsHub/winrm-java

Todas as issues de MetricsHub/winrm-java

Issues semelhantes

Mais issues de Java

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.