Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

[correctness][confirmed-defect][builtins/uniq] uniq ignores an invalid count overwritten by a later flag

Aperta Adatta ai principianti
  • Qualcosa non va?
#663 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
72/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
go, shell
Ambito
cli

Direzione di ricerca

Il flag è memorizzato come una singola stringa in builtins/uniq/uniq.go intorno alla riga 133, e la validazione alla riga 151 controlla solo il valore finale. Leggi la configurazione del flag e quella validazione, poi esegui il riproduttore printf 'a\na\n' | uniq -f wrong -f 0 per vedere il codice di uscita. È completato quando un valore -f precedente non valido fallisce con il messaggio in stile GNU e uscita 1, con uno scenario in tests/scenarios/cmd/uniq/errors/.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Generated by the rshell behavior correctness scanner.

  • Contract: BEH-003
  • Classification: confirmed-defect
  • Impact: low
  • Target: builtins/uniq
  • Scanned commit: 514d1cb8188fda6b1accca5955dfab3b0e8aa34a

An invalid -f value must fail when encountered. Storing only the final value allows -f wrong -f 0 to silently succeed and process input.

Contract sources
  • [SHELL_FEATURES.md:55](https://github.com/datadog/rshell/blob/514d1cb8188fda6b1accca5955dfab3b0e8aa34a/SHELL_FEATURES.md#L55): Supported surface; Bash/GNU behavior applies where no intentional divergence is documented.
Implementation references
  • [builtins/uniq/uniq.go:133](https://github.com/datadog/rshell/blob/514d1cb8188fda6b1accca5955dfab3b0e8aa34a/builtins/uniq/uniq.go#L133): skip-fields is stored as a single StringP value.
  • [builtins/uniq/uniq.go:151](https://github.com/datadog/rshell/blob/514d1cb8188fda6b1accca5955dfab3b0e8aa34a/builtins/uniq/uniq.go#L151): Validation checks only the final stored value.

Fixture setup

Use a fresh empty working directory for each run. Build from /workspace/repo first. No fixture files are needed. Run the CLI and reference commands with that directory as cwd.

Reproducer

printf 'a\na\n' | uniq -f wrong -f 0

Commands used:

  • GOENV_VERSION=1.26.3 timeout 300s go build -o /tmp/zz_correctness_scan_rshell ./cmd/rshell
  • /tmp/zz_correctness_scan_rshell --allow-all-commands --allowed-paths "$PWD" -c 'printf '"'"'a\na\n'"'"' | uniq -f wrong -f 0 '
  • LC_ALL=C.UTF-8 /usr/bin/bash --noprofile --norc -c 'printf '"'"'a\na\n'"'"' | uniq -f wrong -f 0 '
  • GOENV_VERSION=1.26.3 timeout 300s go test ./tests -run '^TestShellScenarios$/^.$/zz_correctness_scan_' -count=2 -v -timeout 300s
Expected

Exit code: 1

stdout:

stderr:

uniq: wrong: invalid number of fields to skip
Actual

Exit code: 0

stdout:

a

stderr:

Verification

  • Reproduction runs: 4
  • Reference: Two fresh CLI fixtures and two fresh scenario fixtures; exploratory runs excluded. Native Bash 5.2.21, GNU coreutils 9.4, grep 3.11, sed 4.9, LC_ALL=C.UTF-8. Docker comparison attempted but image pull returned HTTP 502.
  • Temporary test: tests/scenarios/zz_correctness_scan_uniq_overridden_invalid_count.yaml (removed after evidence capture)
  • Suggested regression-test location: tests/scenarios/cmd/uniq/errors/

Fingerprint: 4f71efcff5df3d86c97abda5e62ade3317a632e940403ad1ba88ab9b1f83be7f

Lingua principale
Go
Stelle
31
Fork
3
Merge medio
2g 9h
PR unite (30g)
14

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di DataDog/rshell

Tutte le issue di DataDog/rshell

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.