Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add proxy build check (by Athena) #100

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Jo0001
Copy link
Member

@Jo0001 Jo0001 commented Sep 22, 2024

grafik

@Jo0001 Jo0001 requested a review from kennytv September 22, 2024 12:30
@kennytv
Copy link
Member

kennytv commented Sep 26, 2024

The code for that should just go into Eduard, cascading web requests in multiple places gets slow really fast

@Jo0001
Copy link
Member Author

Jo0001 commented Sep 26, 2024

The idea was to have it in a central place. But yeah , I'm aware of the performance impact (although I didn't really notice any), so I put it at the end and set a short timeout (which can probably be even lower). 🤷
+I can also try adding caching to Athena (directly in Cloudflare)

@Jo0001
Copy link
Member Author

Jo0001 commented Nov 20, 2024

I tweaked Athena (removed unneeded chain request) and did a test with the normal & tweaked ones:
grafik
Note I did the test from my (shared) home connection so on the server it should be even faster
So I would say with everything below 1sec (which is the configured timeout anyway) its okay

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