askiiart/torznab-toolkit
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix CONFIG == None not working

Browse source
This commit is contained in:
askiiart 2024-11-25 10:20:29 -06:00
parent 52a8c48825
commit 1beab57613
Signed by untrusted user who does not match committer: askiiart
GPG key ID: EA85979611654C30
1 changed files with 13 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
src/api.rs
View file

@ -7,13 +7,12 @@ use rocket::http::Status;
use rocket::response::status;
use xml::writer::{EmitterConfig, XmlEvent};
/// Holds the config for torznab-toolkit.
///
///
/// A search function (`/api?t=search`) and capabilities (`/api?t=caps` - `Caps`) are required, everything else is optional.
///
///
/// <div class="warning">It's required to be set to <i>something</i>, which is why it's an Option set to None.
///
///
/// However, this is NOT optional, and attempting to do anything with CONFIG not set will return an `Err`.</div>
pub static mut CONFIG: Option<Config> = None;
@ -23,11 +22,14 @@ pub static mut CONFIG: Option<Config> = None;
#[get("/api?t=caps")]
pub(crate) fn caps() -> status::Custom<String> {
unsafe {
if CONFIG == None {
return status::Custom(
Status::InternalServerError,
"500 Internal server error: Config not specified".to_string(),
);
match CONFIG {
None => {
return status::Custom(
Status::InternalServerError,
"500 Internal server error: Config not specified".to_string(),
);
}
Some(_) => {}
}
}
@ -121,7 +123,7 @@ mod tests {
CONFIG = Some(create_config());
println!("{:?}", CONFIG);
}
caps();
println!("{:?}", caps());
}
#[test]
@ -129,6 +131,6 @@ mod tests {
unsafe {
println!("{:?}", CONFIG);
}
caps();
println!("{:?}", caps());
}
}