qpismont/herald
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 2 Wiki Activity

Fix missing env var error #5

Merged
qpismont merged 4 commits from 1.0.1 into main 2026-06-16 22:05:25 +02:00
Conversation 5 Commits 4 Files Changed 7
+1 -1
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 3f6c5b5559 - Show all commits
src/env.rs
+1 -1
View File
@@ -41,7 +41,7 @@ pub fn load_config() -> anyhow::Result<EnvConfig> {
}
pub fn try_get_env(key: &str) -> anyhow::Result<String> {
let env = std::env::var(key)?;
let env = std::env::var(key).map_err(|e| anyhow::anyhow!(format!("{}: {}", key, e)))?;
qpismont marked this conversation as resolved Outdated
Herald commented 2026-06-12 23:45:03 +02:00
Outdated
Review
Copy Link
Copy Source

L'utilisation de format! à l'intérieur de anyhow! est redondante. anyhow! accepte directement une chaîne formatée. Préférez anyhow!("{}: {}", key, e) pour plus de clarté et de concision.

L'utilisation de `format!` à l'intérieur de `anyhow!` est redondante. `anyhow!` accepte directement une chaîne formatée. Préférez `anyhow!("{}: {}", key, e)` pour plus de clarté et de concision.
Herald commented 2026-06-12 23:45:03 +02:00
Outdated
Review
Copy Link
Copy Source

Le nom de variable env est trop générique. Envisagez un nom plus explicite comme value ou env_value pour indiquer qu'il s'agit de la valeur de la variable d'environnement, et non de l'environnement lui-même.

Le nom de variable `env` est trop générique. Envisagez un nom plus explicite comme `value` ou `env_value` pour indiquer qu'il s'agit de la valeur de la variable d'environnement, et non de l'environnement lui-même.
Herald commented 2026-06-16 20:54:23 +02:00
Review
Copy Link
Copy Source

Cette transformation en anyhow::Error est claire et appropriée. Bonne pratique pour la propagation d'erreur.

Cette transformation en `anyhow::Error` est claire et appropriée. Bonne pratique pour la propagation d'erreur.
if env.trim().is_empty() {
qpismont marked this conversation as resolved
Herald commented 2026-06-16 19:11:21 +02:00
Review
Copy Link
Copy Source

Ce bloc if ne fait rien si la valeur est vide. Il semble que vous souhaitiez traiter une variable d'environnement vide comme une erreur, mais le code continue et retourne Ok(env_value) à la ligne 50. Vous devriez ajouter une instruction de retour d'erreur (par exemple avec bail! ou return Err(...)) à l'intérieur du bloc if, ou bien utiliser une structure différente pour gérer le cas vide.

Ce bloc `if` ne fait rien si la valeur est vide. Il semble que vous souhaitiez traiter une variable d'environnement vide comme une erreur, mais le code continue et retourne `Ok(env_value)` à la ligne 50. Vous devriez ajouter une instruction de retour d'erreur (par exemple avec `bail!` ou `return Err(...)`) à l'intérieur du bloc `if`, ou bien utiliser une structure différente pour gérer le cas vide.
Herald commented 2026-06-16 20:54:23 +02:00
Review
Copy Link
Copy Source

Après avoir vérifié que la variable d'environnement est vide ou ne contient que des espaces, il n'y a pas de traitement visible (pas de return Err(...) ni de panic!). Si l'intention est de rejeter les valeurs vides, il faut ajouter une gestion d'erreur explicite, par exemple return Err(anyhow::anyhow!("{}: variable vide", key)). Sinon, la fonction continue avec une valeur vide et échouera peut-être plus tard de manière obscure.

Après avoir vérifié que la variable d'environnement est vide ou ne contient que des espaces, il n'y a pas de traitement visible (pas de `return Err(...)` ni de `panic!`). Si l'intention est de rejeter les valeurs vides, il faut ajouter une gestion d'erreur explicite, par exemple `return Err(anyhow::anyhow!("{}: variable vide", key))`. Sinon, la fonction continue avec une valeur vide et échouera peut-être plus tard de manière obscure.
return Err(anyhow!(format!("env var {} is empty", key)));