diff --git a/sasl/Cargo.toml b/sasl/Cargo.toml index 790b8091..2f957112 100644 --- a/sasl/Cargo.toml +++ b/sasl/Cargo.toml @@ -1,8 +1,8 @@ [package] name = "sasl" -version = "0.2.0" +version = "0.3.0" authors = ["lumi "] -description = "A crate for SASL authentication." +description = "A crate for SASL authentication. Currently only does the client side." homepage = "https://gitlab.com/lumi/sasl-rs" repository = "https://gitlab.com/lumi/sasl-rs" documentation = "https://docs.rs/sasl" diff --git a/sasl/src/lib.rs b/sasl/src/lib.rs index ccd6a7c0..9bfcc3e7 100644 --- a/sasl/src/lib.rs +++ b/sasl/src/lib.rs @@ -5,12 +5,12 @@ //! # Examples //! //! ```rust -//! use sasl::{SaslCredentials, SaslMechanism, Error}; +//! use sasl::{Credentials, Mechanism, Error}; //! use sasl::mechanisms::Plain; //! -//! let creds = SaslCredentials::default() -//! .with_username("user") -//! .with_password("pencil"); +//! let creds = Credentials::default() +//! .with_username("user") +//! .with_password("pencil"); //! //! let mut mechanism = Plain::from_credentials(creds).unwrap(); //! @@ -38,40 +38,40 @@ pub use error::Error; /// A struct containing SASL credentials. #[derive(Clone, Debug)] -pub struct SaslCredentials { +pub struct Credentials { /// The requested username. pub username: Option, /// The secret used to authenticate. - pub secret: SaslSecret, + pub secret: Secret, /// Channel binding data, for *-PLUS mechanisms. pub channel_binding: ChannelBinding, } -impl Default for SaslCredentials { - fn default() -> SaslCredentials { - SaslCredentials { +impl Default for Credentials { + fn default() -> Credentials { + Credentials { username: None, - secret: SaslSecret::None, + secret: Secret::None, channel_binding: ChannelBinding::None, } } } -impl SaslCredentials { - /// Creates a new SaslCredentials with the specified username. - pub fn with_username>(mut self, username: N) -> SaslCredentials { +impl Credentials { + /// Creates a new Credentials with the specified username. + pub fn with_username>(mut self, username: N) -> Credentials { self.username = Some(username.into()); self } - /// Creates a new SaslCredentials with the specified password. - pub fn with_password>(mut self, password: P) -> SaslCredentials { - self.secret = SaslSecret::Password(password.into()); + /// Creates a new Credentials with the specified password. + pub fn with_password>(mut self, password: P) -> Credentials { + self.secret = Secret::Password(password.into()); self } - /// Creates a new SaslCredentials with the specified chanel binding. - pub fn with_channel_binding(mut self, channel_binding: ChannelBinding) -> SaslCredentials { + /// Creates a new Credentials with the specified chanel binding. + pub fn with_channel_binding(mut self, channel_binding: ChannelBinding) -> Credentials { self.channel_binding = channel_binding; self } @@ -82,6 +82,8 @@ impl SaslCredentials { pub enum ChannelBinding { /// No channel binding data. None, + /// Advertise that the client does not think the server supports channel binding. + Unsupported, /// p=tls-unique channel binding data. TlsUnique(Vec), } @@ -91,6 +93,7 @@ impl ChannelBinding { pub fn header(&self) -> &[u8] { match *self { ChannelBinding::None => b"n,,", + ChannelBinding::Unsupported => b"y,,", ChannelBinding::TlsUnique(_) => b"p=tls-unique,,", } } @@ -99,6 +102,7 @@ impl ChannelBinding { pub fn data(&self) -> &[u8] { match *self { ChannelBinding::None => &[], + ChannelBinding::Unsupported => &[], ChannelBinding::TlsUnique(ref data) => data, } } @@ -106,7 +110,7 @@ impl ChannelBinding { /// Represents a SASL secret, like a password. #[derive(Clone, Debug, PartialEq, Eq)] -pub enum SaslSecret { +pub enum Secret { /// No extra data needed. None, /// Password required. @@ -114,12 +118,12 @@ pub enum SaslSecret { } /// A trait which defines SASL mechanisms. -pub trait SaslMechanism { +pub trait Mechanism { /// The name of the mechanism. fn name(&self) -> &str; - /// Creates this mechanism from `SaslCredentials`. - fn from_credentials(credentials: SaslCredentials) -> Result where Self: Sized; + /// Creates this mechanism from `Credentials`. + fn from_credentials(credentials: Credentials) -> Result where Self: Sized; /// Provides initial payload of the SASL mechanism. fn initial(&mut self) -> Result, String> { diff --git a/sasl/src/mechanisms/anonymous.rs b/sasl/src/mechanisms/anonymous.rs index 299886b0..b04bc3c8 100644 --- a/sasl/src/mechanisms/anonymous.rs +++ b/sasl/src/mechanisms/anonymous.rs @@ -1,8 +1,8 @@ //! Provides the SASL "ANONYMOUS" mechanism. -use SaslMechanism; -use SaslCredentials; -use SaslSecret; +use Mechanism; +use Credentials; +use Secret; /// A struct for the SASL ANONYMOUS mechanism. pub struct Anonymous; @@ -10,18 +10,18 @@ pub struct Anonymous; impl Anonymous { /// Constructs a new struct for authenticating using the SASL ANONYMOUS mechanism. /// - /// It is recommended that instead you use a `SaslCredentials` struct and turn it into the + /// It is recommended that instead you use a `Credentials` struct and turn it into the /// requested mechanism using `from_credentials`. pub fn new() -> Anonymous { Anonymous } } -impl SaslMechanism for Anonymous { +impl Mechanism for Anonymous { fn name(&self) -> &str { "ANONYMOUS" } - fn from_credentials(credentials: SaslCredentials) -> Result { - if let SaslSecret::None = credentials.secret { + fn from_credentials(credentials: Credentials) -> Result { + if let Secret::None = credentials.secret { Ok(Anonymous) } else { diff --git a/sasl/src/mechanisms/plain.rs b/sasl/src/mechanisms/plain.rs index c80f35e1..fb114338 100644 --- a/sasl/src/mechanisms/plain.rs +++ b/sasl/src/mechanisms/plain.rs @@ -1,8 +1,8 @@ //! Provides the SASL "PLAIN" mechanism. -use SaslMechanism; -use SaslCredentials; -use SaslSecret; +use Mechanism; +use Credentials; +use Secret; /// A struct for the SASL PLAIN mechanism. pub struct Plain { @@ -13,7 +13,7 @@ pub struct Plain { impl Plain { /// Constructs a new struct for authenticating using the SASL PLAIN mechanism. /// - /// It is recommended that instead you use a `SaslCredentials` struct and turn it into the + /// It is recommended that instead you use a `Credentials` struct and turn it into the /// requested mechanism using `from_credentials`. pub fn new, P: Into>(username: N, password: P) -> Plain { Plain { @@ -23,11 +23,11 @@ impl Plain { } } -impl SaslMechanism for Plain { +impl Mechanism for Plain { fn name(&self) -> &str { "PLAIN" } - fn from_credentials(credentials: SaslCredentials) -> Result { - if let SaslSecret::Password(password) = credentials.secret { + fn from_credentials(credentials: Credentials) -> Result { + if let Secret::Password(password) = credentials.secret { if let Some(username) = credentials.username { Ok(Plain::new(username, password)) } diff --git a/sasl/src/mechanisms/scram.rs b/sasl/src/mechanisms/scram.rs index e1fa8213..4296bc86 100644 --- a/sasl/src/mechanisms/scram.rs +++ b/sasl/src/mechanisms/scram.rs @@ -2,9 +2,9 @@ use base64; -use SaslMechanism; -use SaslCredentials; -use SaslSecret; +use Mechanism; +use Credentials; +use Secret; use ChannelBinding; use error::Error; @@ -147,7 +147,7 @@ impl Scram { /// Constructs a new struct for authenticating using the SASL SCRAM-* and SCRAM-*-PLUS /// mechanisms, depending on the passed channel binding. /// - /// It is recommended that instead you use a `SaslCredentials` struct and turn it into the + /// It is recommended that instead you use a `Credentials` struct and turn it into the /// requested mechanism using `from_credentials`. pub fn new, P: Into>(username: N, password: P, channel_binding: ChannelBinding) -> Result, Error> { Ok(Scram { @@ -177,13 +177,13 @@ impl Scram { } } -impl SaslMechanism for Scram { +impl Mechanism for Scram { fn name(&self) -> &str { // TODO: this is quite the workaround… &self.name } - fn from_credentials(credentials: SaslCredentials) -> Result, String> { - if let SaslSecret::Password(password) = credentials.secret { + fn from_credentials(credentials: Credentials) -> Result, String> { + if let Secret::Password(password) = credentials.secret { if let Some(username) = credentials.username { Scram::new(username, password, credentials.channel_binding) .map_err(|_| "can't generate nonce".to_owned()) @@ -284,7 +284,7 @@ impl SaslMechanism for Scram { #[cfg(test)] mod tests { - use ::SaslMechanism; + use ::Mechanism; use super::*;