Add more tests against ss link parser

ytflow-app-util/Cargo.toml

@@ -15,4 +15,5 @@ base64 = "0.21"
 thiserror = "1"
 cbor4ii = { version = "0.3", features = ["use_std", "serde1"] }
 serde = { version = "1", features = ["derive"] }
+serde_bytes = "0.11"
 ytflow = { path = "../ytflow" }

ytflow-app-util/src/proxy/protocol/shadowsocks.rs

@@ -1,7 +1,9 @@
+use serde_bytes::ByteBuf;
+
 use ytflow::plugin::shadowsocks::SupportedCipher;
 
 #[derive(Debug, Clone, PartialEq, Eq)]
 pub struct ShadowsocksProxy {
     pub cipher: SupportedCipher,
-    pub password: Vec<u8>,
+    pub password: ByteBuf,
 }

ytflow-app-util/src/share_link/decode.rs

@@ -18,10 +18,12 @@ pub static BASE64_ENGINE: base64::engine::GeneralPurpose = base64::engine::Gener
 pub enum DecodeError {
     #[error("invalid URL")]
     InvalidUrl,
-    #[error("invalid URL or base64 encoding")]
+    #[error("invalid URL, UTF-8 or Base64 encoding")]
     InvalidEncoding,
-    #[error("invalid value")]
-    InvalidValue,
+    #[error(r#""{0}" is required, but is missing"#)]
+    MissingInfo(&'static str),
+    #[error(r#"unknown value for field "{0}"#)]
+    UnknownValue(&'static str),
     #[error("unknown URL scheme")]
     UnknownScheme,
     #[error(r#"extra parameter "{0}""#)]
@@ -33,7 +35,7 @@ pub type DecodeResult<T> = Result<T, DecodeError>;
 pub(super) type QueryMap<'a> = BTreeMap<Cow<'a, str>, Cow<'a, str>>;
 
 pub fn decode_share_link(link: &str) -> Result<Proxy, DecodeError> {
-    let url = url::Url::parse(link).map_err(|_| DecodeError::InvalidUrl)?;
+    let url = url::Url::parse(link.trim()).map_err(|_| DecodeError::InvalidUrl)?;
     let mut queries = url.query_pairs().collect::<QueryMap>();
 
     let proxy = match url.scheme() {