Detect loops in entity resolution

Aperta
#945 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à
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
35/100
Tipo di issue
Funzionalità
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
rust
Ambito
backend

Direzione di ricerca

Start by reading the EntityResolver trait and the entity expansion paths it serves; the issue says loop detection must apply to every entity expansion and be coupled to that trait. Decide how immediate loop detection relates to recursion-limit errors, then map the affected expansion paths and tests. Done means loops are detected consistently across entity expansion and the resulting error behavior is covered by tests.

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

Descrizione

enhancement help wanted

Thoughts on automatic loop detection? (note from previous PR)

Also I've noticed that some implementations detect an entity loop immediately instead of processing until the recursion limit is reached. Should that be two separate errors in your opinion, or one error?

Obviously it would not need to be added in this PR. Just, as a general concept, should an issue be filed for it

Originally posted by @dralley in https://github.com/tafia/quick-xml/pull/943#discussion_r2835819274

The problem, is that loop detection should be applied to any entity expansion, and so should be highly coupled with the EntityResolver trait.

Lingua principale
Rust
Stelle
1.6k
Fork
299
Merge medio
2g 4h
PR unite (30g)
9

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 tafia/quick-xml

Tutte le issue di tafia/quick-xml

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.