GeneralBots/botserver
2
1
Fork 0
Code Activity Actions

fix: Replace Runtime::new().block_on() with thread::spawn in AuthConfig
Some checks failed
BotServer CI/CD / build (push) Failing after 1m14s
Details

Browse source 
- AuthConfig::from_env() was creating a new Runtime and calling block_on
  directly, causing panic when main tokio runtime is already active
- Now uses std:🧵:spawn + new_current_thread().block_on() pattern
- Follows AGENTS.md pattern for async-from-sync bridges
This commit is contained in:
Rodrigo Rodriguez (Pragmatismo) 2026-04-04 08:25:43 -03:00
parent c0c06e92eb
commit 0d3cfbe0f7
1 changed files with 27 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

47
src/security/auth_api/config.rs
View file

@ -54,26 +54,33 @@ impl AuthConfig {
if let Ok(secret) = std::env::var("VAULT_TOKEN") {
if !secret.is_empty() {
let rt = tokio::runtime::Runtime::new().ok();
if let Some(rt) = rt {
let sm = crate::core::shared::utils::get_secrets_manager_sync();
if let Some(sm) = sm {
if let Ok(secrets) =
rt.block_on(sm.get_secret(crate::core::secrets::SecretPaths::JWT))
{
if let Some(s) = secrets.get("secret") {
config.jwt_secret = Some(s.clone());
}
if let Some(r) = secrets.get("require_auth") {
config.require_auth = r == "true" || r == "1";
}
if let Some(p) = secrets.get("anonymous_paths") {
config.allow_anonymous_paths = p
.split(',')
.map(|s| s.trim().to_string())
.filter(|s| !s.is_empty())
.collect();
}
let sm = crate::core::shared::utils::get_secrets_manager_sync();
if let Some(sm) = sm {
let (tx, rx) = std::sync::mpsc::channel();
std::thread::spawn(move || {
let rt = tokio::runtime::Builder::new_current_thread()
.enable_all()
.build();
let result = if let Ok(rt) = rt {
rt.block_on(async { sm.get_secret(crate::core::secrets::SecretPaths::JWT).await })
} else {
Err("Failed to create runtime".into())
};
let _ = tx.send(result);
});
if let Ok(Ok(secrets)) = rx.recv() {
if let Some(s) = secrets.get("secret") {
config.jwt_secret = Some(s.clone());
}
if let Some(r) = secrets.get("require_auth") {
config.require_auth = r == "true" || r == "1";
}
if let Some(p) = secrets.get("anonymous_paths") {
config.allow_anonymous_paths = p
.split(',')
.map(|s| s.trim().to_string())
.filter(|s| !s.is_empty())
.collect();
}
}
}