External pager resets custom SIGINT handlers to SIG_DFL

Aperta Adatta ai principianti
  • Qualcosa non va?
#723 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

@hugosmoreira ci sta già lavorando.

Dal 26/10/2026.

  • #725 di @hugosmoreira — aperta

Valutazione

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

Direzione di ricerca

Inizia con More() in fire/console/console_io.py e verifica come viene eseguito il pager esterno e come vengono gestite le eccezioni. Salva il gestore attivo prima che SIGINT venga ignorato, poi assicurati che venga ripristinato dopo l’uscita del pager, anche in caso di eccezioni. Il lavoro è terminato quando il gestore precedente resta installato in entrambi i casi; l’issue non fornisce un file di test, quindi individua un test adatto per questo comportamento.

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

Descrizione

In fire/console/console_io.py, the More() function temporarily ignores SIGINT while an external pager runs.

However, it restores SIG_DFL instead of the handler previously installed by the application.

Problematic code
signal.signal(signal.SIGINT, signal.SIG_IGN)

# Execute external pager

signal.signal(signal.SIGINT, signal.SIG_DFL)
Expected behavior

The function should preserve the existing signal handler and restore it after the pager exits, including when an exception occurs.

Actual behavior

Based on the code, applications using custom SIGINT handlers lose their original handler after pager execution. Exceptions may also leave SIGINT ignored.

Suggested fix

Save the original handler and restore it in a finally block.

Testing

Identified through source-code inspection. An interactive pager reproduction has not yet been executed.

Lingua principale
Python
Stelle
28.2k
Fork
1.5k
Nessuna PR unita negli ultimi 30g

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 google/python-fire

Tutte le issue di google/python-fire

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.