Merge #786: Update error message when parsing DescriptorPublicKey

apoelstra · apoelstra · commit 772d7de02daa · 2024-12-20T04:15:55.000Z
7f7f547 Update error message when parsing DescriptorPublicKey (Sanket Kanjalkar) Pull request description: Sorry if there are lint issues, I can try fixing them later. I am directly editing from github web UI ACKs for top commit: brunoerg: ACK 7f7f547 apoelstra: ACK 7f7f547; successfully ran local tests; looks like linter is happy Tree-SHA512: b2a8896d911eedb72eb646a9d3a5176e47441c5c8d58e909c4a4029e16dac3a64309057f08eae024bb7d4369cc058595f224982f29f80dd67a21850b3262cb7d
diff --git a/src/descriptor/key.rs b/src/descriptor/key.rs
@@ -486,7 +486,8 @@ impl FromStr for DescriptorPublicKey {
         // A "raw" public key without any origin is the least we accept.
         if s.len() < 64 {
             return Err(DescriptorKeyParseError(
-                "Key too short (<66 char), doesn't match any format",
+                "Key too short (<64 characters); use parse_descriptor for parsing \
+                 descriptors with private keys.",
             ));
         }
 

Original file line number	Diff line number	Diff line change
`@@ -486,7 +486,8 @@ impl FromStr for DescriptorPublicKey {`
`486`	`486`	`// A "raw" public key without any origin is the least we accept.`
`487`	`487`	`if s.len() < 64 {`
`488`	`488`	`return Err(DescriptorKeyParseError(`
`489`		`- "Key too short (<66 char), doesn't match any format",`
	`489`	`+ "Key too short (<64 characters); use parse_descriptor for parsing \`
	`490`	`+ descriptors with private keys.",`
`490`	`491`	`));`
`491`	`492`	`}`
`492`	`493`