clippy, small documentation fixup

OpenXbox · Dec 21, 2023 · b80861d · b80861d
1 parent f5b0ea4
commit b80861d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/src/authenticator.rs b/src/authenticator.rs
@@ -410,10 +410,7 @@ impl XalAuthenticator {
     pub fn oauth_client(&self) -> Result<BasicClient, Error> {
         let client = OAuthClient::new(
             ClientId::new(self.app_params.client_id.to_string()),
-            self.app_params
-                .client_secret
-                .clone()
-                .map(|x| ClientSecret::new(x)),
+            self.app_params.client_secret.clone().map(ClientSecret::new),
             AuthUrl::new(Constants::OAUTH20_AUTHORIZE_URL.to_string())?,
             Some(TokenUrl::new(Constants::OAUTH20_TOKEN_URL.to_string())?),
         )

diff --git a/src/models.rs b/src/models.rs
@@ -506,9 +506,9 @@ impl ToString for DeviceType {
 /// Mandatory for XAL authentication flow
 #[derive(Debug, Serialize, Deserialize, PartialEq, Eq, Clone)]
 pub struct XalAppParameters {
-    /// App Id (For authorization/permission scope)
+    /// OAuth2 Client Id
     pub client_id: String,
-    /// App Title-Id (For TitleToken)
+    /// App Title-Id (Required for SISU auth flow, for TitleToken)
     pub title_id: Option<String>,
     /// Scopes
     pub auth_scopes: Vec<Scope>,