solana-remote-wallet does not intialize app version correctly

[cavemanloverboy]

    pub fn new(device: hidapi::HidDevice) -> Self {
        Self {
            device,
            pretty_path: String::default(),
            version: FirmwareVersion::new(0, 0, 0),
        }
    }

is problematic due to the use of

    fn outdated_app(&self) -> bool {
        self.version < DEPRECATE_VERSION_BEFORE
    }

in determining which logic to use. Presently, sign_message does not work with message above the chunk size of 255 on 1.4.1 because the logic for oudated_app is used.

This problem can be fixed by calling get_configuration_vector and reading the last 3 bytes, or get_firmware_version

[cavemanloverboy]

Unclear if we want to break api to fix or introduce new_with_version which returns a Result:

pub fn new_with_version(device: hidapi::HidDevice) -> Result<Self, RemoteWalletError> {
    let mut slf = Self {
        device,
        pretty_path: String::default(),
        version: FirmwareVersion::new(0, 0, 0),
    };
    slf.version = slf.get_firmware_version()?;
    Ok(slf)
}

[cavemanloverboy]

let me know which way you want to go, western man