citadel-tech/coinswap

Password handler remove I/O operation

Geschlossen

#956 geöffnet am 27.07.2026

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (132 Forks)auto 404
enhancementgood first issue

Repository-Metriken

Stars
 (159 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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.

Contributor Guide

Research-Richtung
Finden Sie die Funktion `load sensitive struct`, entfernen Sie den `prompt password` Aufruf und stellen Sie sicher, dass der Aufrufer den Fall eines None Passworts angemessen behandelt.
Tech Stack
rust
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Rust
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70