Merge pull request #1694 from ehuss/rate-limit-error-check

Mark-Simulacrum · web-flow · commit 79690e4dc0d7 · 2023-06-13T16:57:05.000-04:00
Check for error response when fetching the rate limit
diff --git a/src/github.rs b/src/github.rs
@@ -124,6 +124,7 @@ impl GithubClient {
                         .unwrap(),
                 )
                 .await?;
+            rate_resp.error_for_status_ref()?;
             let rate_limit_response = rate_resp.json::<RateLimitResponse>().await?;
 
             // Check url for search path because github has different rate limits for the search api

Original file line number	Diff line number	Diff line change
`@@ -124,6 +124,7 @@ impl GithubClient {`
`124`	`124`	`.unwrap(),`
`125`	`125`	`)`
`126`	`126`	`.await?;`
	`127`	`+ rate_resp.error_for_status_ref()?;`
`127`	`128`	`let rate_limit_response = rate_resp.json::<RateLimitResponse>().await?;`
`128`	`129`
`129`	`130`	`// Check url for search path because github has different rate limits for the search api`