Update docs, remove panic in favor of Err Result
This commit is contained in:
parent
c436fdb857
commit
5031db6547
15
src/limit.rs
15
src/limit.rs
|
@ -57,7 +57,8 @@ impl LimitedRequester {
|
||||||
|
|
||||||
## Errors
|
## Errors
|
||||||
|
|
||||||
This method will panic, if:
|
This method will error, if:
|
||||||
|
- The request does not return a success status code (200-299)
|
||||||
- The supplied [`RequestBuilder`](reqwest::RequestBuilder) contains invalid or incomplete
|
- The supplied [`RequestBuilder`](reqwest::RequestBuilder) contains invalid or incomplete
|
||||||
information
|
information
|
||||||
- There has been an error with processing (unwrapping) the [`Response`](`reqwest::Response`)
|
- There has been an error with processing (unwrapping) the [`Response`](`reqwest::Response`)
|
||||||
|
@ -72,9 +73,15 @@ impl LimitedRequester {
|
||||||
user_rate_limits: &mut Limits,
|
user_rate_limits: &mut Limits,
|
||||||
) -> Result<Response, InstanceServerError> {
|
) -> Result<Response, InstanceServerError> {
|
||||||
if self.can_send_request(limit_type, instance_rate_limits, user_rate_limits) {
|
if self.can_send_request(limit_type, instance_rate_limits, user_rate_limits) {
|
||||||
let built_request = request
|
let built_request = match request.build() {
|
||||||
.build()
|
Ok(request) => request,
|
||||||
.unwrap_or_else(|e| panic!("Error while building the Request for sending: {}", e));
|
Err(e) => {
|
||||||
|
return Err(InstanceServerError::RequestErrorError {
|
||||||
|
url: "".to_string(),
|
||||||
|
error: e.to_string(),
|
||||||
|
})
|
||||||
|
}
|
||||||
|
};
|
||||||
let result = self.http.execute(built_request).await;
|
let result = self.http.execute(built_request).await;
|
||||||
let response = match result {
|
let response = match result {
|
||||||
Ok(is_response) => is_response,
|
Ok(is_response) => is_response,
|
||||||
|
|
Loading…
Reference in New Issue