citadel-tech/coinswap

Password handler remove I/O operation

Chiusa

#956 aperta il 27 lug 2026

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Rust (132 fork)auto 404
enhancementgood first issue

Metriche repository

Star
 (159 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Description

In load_sensitive_struct, in the case of a None password, if parsing fails we ask the user to enter the password utill::prompt_password("Enter encryption passphrase: ".to_string())?, which is irrelevant to this API — it's a terminal I/O operation that doesn't belong in a decrypt helper and should be handled by the caller if the parse fails.

Guida contributor

Direzione di ricerca
Individua la funzione `load sensitive struct`, rimuovi la chiamata a `prompt password` e assicurati che il chiamante gestisca correttamente il caso in cui la password sia None.
Tech stack
rust
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70