diff options
author | Robin Krahl <robin.krahl@ireas.org> | 2019-01-28 14:24:12 +0000 |
---|---|---|
committer | Robin Krahl <robin.krahl@ireas.org> | 2019-02-02 10:25:02 +0000 |
commit | 0972bbe82623c3d9649b6023d8f50d304aa0cde6 (patch) | |
tree | 5f9c2a3cd1981ae2b04cd07ba466f2a8d8f74f0c /src/lib.rs | |
parent | f49e61589e32217f97c94aa86d826f6b65170fba (diff) | |
download | nitrokey-rs-0972bbe82623c3d9649b6023d8f50d304aa0cde6.tar.gz nitrokey-rs-0972bbe82623c3d9649b6023d8f50d304aa0cde6.tar.bz2 |
Refactor User and Admin to use a mutable reference
In the initial nitrokey-rs implementation, the Admin and the User struct
take the Device by value to make sure that the user cannot initiate a
second authentication while this first is still active (which would
invalidate the temporary password). Now we realized that this is not
necessary – taking a mutable reference has the same effect, but leads to
a much cleaner API.
This patch refactors the Admin and User structs – and all dependent code
– to use a mutable reference instead of a Device value.
Diffstat (limited to 'src/lib.rs')
-rw-r--r-- | src/lib.rs | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -44,7 +44,7 @@ //! # use nitrokey::Error; //! //! # fn try_main() -> Result<(), Error> { -//! let device = nitrokey::connect()?; +//! let mut device = nitrokey::connect()?; //! let slot_data = OtpSlotData::new(1, "test", "01234567890123456689", OtpMode::SixDigits); //! match device.authenticate_admin("12345678") { //! Ok(mut admin) => { @@ -53,7 +53,7 @@ //! Err(err) => eprintln!("Could not write slot: {}", err), //! } //! }, -//! Err((_, err)) => eprintln!("Could not authenticate as admin: {}", err), +//! Err(err) => eprintln!("Could not authenticate as admin: {}", err), //! } //! # Ok(()) //! # } |