From 914f69c155125f4f313948a3673586a4f376c3c6 Mon Sep 17 00:00:00 2001
From: Dominik George <nik@naturalnet.de>
Date: Thu, 6 May 2021 00:48:25 +0200
Subject: [PATCH] [PAM] Remove unnecessary reqwest error checks
It seems all 4xx errors are pushed into ServerResponse
---
src/pam.rs | 9 ++-------
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/src/pam.rs b/src/pam.rs
index f85ab50..c6e3eb8 100644
--- a/src/pam.rs
+++ b/src/pam.rs
@@ -85,13 +85,8 @@ fn do_legacy_auth(username: String, password: String, config: Config) -> Result<
Err(e) => match e {
RequestTokenError::Request(re) => match re {
reqwest::Error::Reqwest(ree) => {
- if ree.is_status() {
- error!("Authentication failed for invalid grant: {}", ree);
- return Err(PamError::AUTH_ERR);
- } else {
- error!("Request error requesting token: {}", ree);
- return Err(PamError::AUTHINFO_UNAVAIL);
- }
+ error!("Request error requesting token: {}", ree);
+ return Err(PamError::AUTHINFO_UNAVAIL);
},
reqwest::Error::Http(he) => {
error!("HTTP error requesting token: {}", he);
--
GitLab