https://github.com/cachebag/nmrs/blob/a031a22b3fd38d2417645f416d3dd51d0d41fdcf/nmrs/src/core/connection.rs#L635-L652
The logic for empty PSK passwords is unclear and could lead to user confusion. When a user provides an empty PSK and there's no saved connection, we return NoSavedConnection error, but the real issue is that they didn't provide a password.
We should probably add a new error variant ConnectionError::MissingPassword or ConnectionError::InvalidCredentials for clarity.
https://github.com/cachebag/nmrs/blob/a031a22b3fd38d2417645f416d3dd51d0d41fdcf/nmrs/src/core/connection.rs#L635-L652
The logic for empty PSK passwords is unclear and could lead to user confusion. When a user provides an empty PSK and there's no saved connection, we return
NoSavedConnectionerror, but the real issue is that they didn't provide a password.We should probably add a new error variant
ConnectionError::MissingPasswordorConnectionError::InvalidCredentialsfor clarity.