morrigan/crabbysearch
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #31 from alamin655/page_url

Browse source 
Fix page_url assignment in search route
This commit is contained in:
neon_arch 2023-05-20 08:34:46 +00:00 • committed by GitHub
commit 5f052dbee8
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
src/server/routes.rs
View file

@ -81,11 +81,10 @@ pub async fn search(
.insert_header(("location", "/")) .insert_header(("location", "/"))
.finish()) .finish())
} else { } else {
// Initialize the page url as an empty string let page_url: String; // Declare the page_url variable without initializing it
let mut page_url = String::new();
// Find whether the page is valid page number if not then return // ...
// the first page number and also construct the page_url accordingly
let page = match params.page { let page = match params.page {
Some(page_number) => { Some(page_number) => {
if page_number <= 1 { if page_number <= 1 {
@ -99,7 +98,7 @@ pub async fn search(
"http://{}:{}/search?q={}&page={}", "http://{}:{}/search?q={}&page={}",
config.binding_ip_addr, config.port, query, page_number config.binding_ip_addr, config.port, query, page_number
); );
page_number page_number
} }
} }
@ -111,11 +110,11 @@ pub async fn search(
req.uri(), req.uri(),
1 1
); );
1 1
} }
}; };
// fetch the cached results json. // fetch the cached results json.
let cached_results_json = redis_cache.cached_results_json(&page_url); let cached_results_json = redis_cache.cached_results_json(&page_url);
// check if fetched results was indeed fetched or it was an error and if so // check if fetched results was indeed fetched or it was an error and if so