Exception str does not need to be translated
What the developer wrote
Authored by kdmukai
Exception str does not need to be translated
What changed, and why it matters
This commit removes translation markup from an internal error message. It is a code-quality cleanup with no security relevance. The exception is raised when the program reaches an impossible state, and the message is not shown to users in normal operation.
No security action needed. This is a routine localization/code-quality change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The change replaces raise Exception(_("Can't validate a single sig addr without specifying a seed")) with raise Exception("Can't validate a single sig addr without specifying a seed") and adds a comment that this path should be unreachable. The _() function marks text for internationalization/translation. Removing it from an exception string means the message will not be translated, which is intentional because exception strings are typically for debugging, not user-facing UI. There is no functional, cryptographic, or security behavior change.
Changed components
src/seedsigner/views/seed_views.pyInspect captured patch +2 / −1
diff --git a/src/seedsigner/views/seed_views.py b/src/seedsigner/views/seed_views.py
index 8093318..dc8a543 100644
--- a/src/seedsigner/views/seed_views.py
+++ b/src/seedsigner/views/seed_views.py
@@ -1868,7 +1868,8 @@ class SeedAddressVerificationView(View):
self.seed_derivation_override = ""
if not self.is_multisig:
if seed_num is None:
- raise Exception(_("Can't validate a single sig addr without specifying a seed"))
+ # Shouldn't be able to get here
+ raise Exception("Can't validate a single sig addr without specifying a seed")
self.seed_num = seed_num
self.seed = self.controller.get_seed(seed_num)
self.seed_derivation_override = self.seed.derivation_override(sig_type=SettingsConstants.SINGLE_SIG)
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.