Skip to content

Commit

Permalink
refactor: clean lint warning messages
Browse files Browse the repository at this point in the history
  • Loading branch information
Manuel Sopena Ballesteros committed Oct 28, 2024
1 parent 3aa22a4 commit f86a47d
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 66 deletions.
6 changes: 3 additions & 3 deletions src/common/jwt_ops.rs
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,11 @@ use serde_json::Value;

use crate::hsm;

// FIXME: replace Error to my own one
/* // FIXME: replace Error to my own one
#[deprecated(
note = "Please, avoid using this function, if you need to get the list of HSM groups available to the user, then use `mesa::common::jwt_ops::get_hsm_name_available` because this function has the hack removing system wide hsm group names like alps, aplsm, alpse, etc. If you want the preffereed username, then use `mesa::common::jwt_ops::`mesa::common::jwt_ops::get_preferred_username"
)]
pub fn get_claims_from_jwt_token(token: &str) -> Result<Value, Box<dyn Error>> {
)] */
fn get_claims_from_jwt_token(token: &str) -> Result<Value, Box<dyn Error>> {
let base64_claims = token
.split(' ')
.nth(1)
Expand Down
56 changes: 0 additions & 56 deletions src/keycloak.rs

This file was deleted.

1 change: 0 additions & 1 deletion src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,5 @@ pub mod error;
#[cfg(feature = "ochami")]
pub mod hsm;
pub mod ims;
pub mod keycloak;
pub mod node;
pub mod pcs;
8 changes: 4 additions & 4 deletions src/node/console.rs
Original file line number Diff line number Diff line change
Expand Up @@ -119,8 +119,8 @@ pub async fn get_container_attachment_to_cfs_session_image_target(

// Waiting for pod to start
while pods.items.is_empty() && i <= max {
format!(
"\nPod for cfs session {} not ready. Trying again in 2 secs. Attempt {} of {}\n",
println!(
"Pod for cfs session {} not ready. Trying again in 2 secs. Attempt {} of {}",
cfs_session_name,
i + 1,
max
Expand Down Expand Up @@ -192,8 +192,8 @@ pub async fn get_container_attachment_to_cfs_session_image_target(

// Waiting for pod to start
while pods.items.is_empty() && i <= max {
format!(
"\nPod for cfs session {} not ready. Trying again in 2 secs. Attempt {} of {}\n",
println!(
"Pod for cfs session {} not ready. Trying again in 2 secs. Attempt {} of {}",
cfs_session_name,
i + 1,
max
Expand Down
2 changes: 0 additions & 2 deletions src/pcs.rs
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,6 @@ pub mod transitions {
shasta_base_url,
shasta_token,
shasta_root_cert,
xname_vec,
transition_id,
)
.await?;
Expand All @@ -291,7 +290,6 @@ pub mod transitions {
shasta_base_url: &str,
shasta_token: &str,
shasta_root_cert: &[u8],
xname_vec: &Vec<String>,
transition_id: &str,
) -> Result<Value, Error> {
let mut transition_status = "";
Expand Down

0 comments on commit f86a47d

Please sign in to comment.