Skip to content

fix: Make Google request single-threaded - #213

Merged
iphydf merged 1 commit into
TokTok:masterfrom
nickolay168:nickolay168/add_google_lock
Sep 13, 2026
Merged

iphydf merged 1 commit into
TokTok:masterfrom
nickolay168:nickolay168/add_google_lock

Conversation

@nickolay168

@nickolay168 nickolay168 commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Making the requests to google effectively single-threaded and also adding retry for internal server errors (500). It is another attempt to fix throttling of CI/CD.


This change is Reviewable

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Tip

Preview URL:

@github-actions github-actions Bot added the bug Bug fix for the user, not a fix to a build script label Sep 9, 2026
@nickolay168
nickolay168 force-pushed the nickolay168/add_google_lock branch from 4229515 to c10ba65 Compare September 9, 2026 07:23
@nickolay168
nickolay168 force-pushed the nickolay168/add_google_lock branch from c10ba65 to 7dbf334 Compare September 9, 2026 07:26
@nickolay168
nickolay168 force-pushed the nickolay168/add_google_lock branch from 7dbf334 to d845b38 Compare September 9, 2026 07:28
@nickolay168
nickolay168 marked this pull request as ready for review September 9, 2026 07:33
@nickolay168
nickolay168 force-pushed the nickolay168/add_google_lock branch from d845b38 to 37f6168 Compare September 12, 2026 07:48
@nickolay168
nickolay168 force-pushed the nickolay168/add_google_lock branch from 37f6168 to cf8e888 Compare September 12, 2026 08:04
@Green-Sky

Copy link
Copy Markdown
Member

Is the google api limit really that bad?

@nickolay168

Copy link
Copy Markdown
Contributor Author

Is the google api limit really that bad?

I am constantly seeing this error on the PR https://github.com/TokTok/qTox/actions/runs/34313138125/job/102343745673?pr=750
I used to run the same scripts locally, and it has passed though.

@nickolay168

Copy link
Copy Markdown
Contributor Author

@Green-Sky it looks like we have another problem, Baidu started to require authorization token.

@iphydf iphydf added this to the v0.8.5 milestone Sep 13, 2026
@iphydf
iphydf merged commit aa5514e into TokTok:master Sep 13, 2026
29 checks passed

This branch was successfully deployed

1 active deployment
netlify aa5514e2 Deployed Sep 13, 2026 by iphydf via netlify / deploy #403
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug fix for the user, not a fix to a build script

Development

Successfully merging this pull request may close these issues.

3 participants