Skip to content

github: Log graphql ratelimit for debug - #219

Merged
jerry-skydio merged 1 commit into
Skydio:mainfrom
brian-kubisiak-skydio:brian/revup/main/ratelimit-info
Oct 17, 2025
Merged

github: Log graphql ratelimit for debug#219
jerry-skydio merged 1 commit into
Skydio:mainfrom
brian-kubisiak-skydio:brian/revup/main/ratelimit-info

Conversation

@brian-kubisiak-skydio

Copy link
Copy Markdown

Ratelimit information is already available in the HTTP response
headers for graphql requests; log this information for debug.

Signed-off-by: Brian Kubisiak brian@kubisiak.com

@brian-kubisiak-skydio

Copy link
Copy Markdown
Author

Reviews in this chain:
#219 github: Log graphql ratelimit for debug

@brian-kubisiak-skydio

brian-kubisiak-skydio commented Oct 17, 2025

Copy link
Copy Markdown
Author
# head base diff date summary
0 ec6b08d3 d29851a6 diff Oct 17 6:23 AM 1 file changed, 9 insertions(+)
1 d8263642 d29851a6 diff Oct 17 6:51 AM 1 file changed, 8 insertions(+), 3 deletions(-)
2 4dd24fa7 d29851a6 diff Oct 17 6:56 AM 1 file changed, 3 insertions(+), 3 deletions(-)
3 954a30af d29851a6 diff Oct 17 7:24 AM 1 file changed, 1 insertion(+), 3 deletions(-)

@brian-kubisiak-skydio
brian-kubisiak-skydio force-pushed the brian/revup/main/ratelimit-info branch 2 times, most recently from d826364 to 4dd24fa Compare October 17, 2025 13:56
Ratelimit information is already available in the HTTP response
headers for graphql requests; log this information for debug.

Signed-off-by: Brian Kubisiak <brian@kubisiak.com>
@brian-kubisiak-skydio
brian-kubisiak-skydio force-pushed the brian/revup/main/ratelimit-info branch from 4dd24fa to 954a30a Compare October 17, 2025 14:24
@jerry-skydio
jerry-skydio merged commit 1a70703 into Skydio:main Oct 17, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants