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

darnit plan and darnit validate crash on openssf-baseline: controls mix level=None and int

Aperta
  • Qualcosa non va?
#568 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@i-am-paradox ci sta già lavorando.

Dal 13/10/2026.

  • #597 di @i-am-paradox — aperta

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
45/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
python
Ambito
cli

Direzione di ricerca

The crash is in cmd_plan in darnit/cli.py (around line 425, where by_level keys are sorted) and in the matching sort in darnit validate. Check how ControlConfig.level and tags.level are read, and FrameworkConfig.get_controls_by_level, before choosing a canonical source as framework-design.md should specify. Done means darnit plan and darnit validate run on openssf-baseline.toml without a TypeError, with None sorted last, and a test covers every shipped framework TOML.

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

Descrizione

interface-cli module-baseline module-core needs triage

darnit plan and darnit validate crash on the shipped openssf-baseline framework (reproduced on main 06e905e).

$ darnit plan
  File ".../darnit/cli.py", line 425, in cmd_plan
    for level in sorted(by_level.keys()):
TypeError: '<' not supported between instances of 'int' and 'NoneType'

$ darnit validate packages/darnit-baseline/src/darnit_baseline/openssf-baseline.toml
    logger.info(f"  By level: {', '.join(f'L{k}={v}' for k, v in sorted(by_level.items()))}")
TypeError: '<' not supported between instances of 'int' and 'NoneType'

Cause: both commands group controls by ControlConfig.level. openssf-baseline declares levels in tags.level (tags = { level = 1, ... }), so 65 of its 66 controls have level = None. One control, STAGE1-REF-SECURITY-01, sets the top-level level, so the keys mix int and None and sorted() raises. darnit plan -f community-spec works because none of its controls set level.

Same root cause, no crash: FrameworkConfig.get_controls_by_level(n) filters on ControlConfig.level, so it returns nothing for the 65 baseline controls that use tags.level. It has no production caller today. The audit path reads the level some other way and is not affected; a fix should check that.

Suggested direction:

  1. Give a control's level one source. Either ControlConfig.level falls back to tags.level, or the shipped TOMLs move to the top-level field. The spec (framework-design.md) should say which one is canonical.
  2. Make plan and validate sort with None last instead of crashing.
  3. Add a test that runs plan and validate against every shipped framework TOML.

Found while removing dead code for #487 (#566).

Drafted with Claude Code; reviewed and posted by me.

Lingua principale
Python
Stelle
22
Fork
27
Merge medio
1g 20h
PR unite (30g)
39

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 darnitdevorg/darnit

Tutte le issue di darnitdevorg/darnit

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.