Skip to content

Commit

Permalink
[ingress] Parse the value of delaySec query param (#1498)
Browse files Browse the repository at this point in the history
This commit fixes a bug, where we previously tried parsing the
queryParam name (key) instead of its value.
  • Loading branch information
igalshilman authored May 6, 2024
1 parent 33455b3 commit 46d3f67
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion crates/ingress-http/src/handler/service_handler.rs
Original file line number Diff line number Diff line change
Expand Up @@ -328,7 +328,7 @@ fn parse_delay(query: Option<&str>) -> Result<Option<Duration>, HandlerError> {
}
if k.eq_ignore_ascii_case(DELAYSEC_QUERY_PARAM) {
return Ok(Some(Duration::from_secs(
k.parse().map_err(HandlerError::BadDelaySecDuration)?,
v.parse().map_err(HandlerError::BadDelaySecDuration)?,
)));
}
}
Expand Down

0 comments on commit 46d3f67

Please sign in to comment.