Skip to content

Conversation

sandhose
Copy link
Member

We were almost systematically parsing the user-agent, even though we did not do anything with the parsed data.
We also compiled regexes every time, which is expensive in terms of CPU.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Apr 24, 2025

Deploying matrix-authentication-service-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0cfea60
Status: ✅  Deploy successful!
Preview URL: https://dd0aa0dd.matrix-authentication-service-docs.pages.dev
Branch Preview URL: https://quenting-perf-useragent.matrix-authentication-service-docs.pages.dev

View logs

@sandhose sandhose requested a review from reivilibre April 24, 2025 11:40
@sandhose sandhose added the T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. label Apr 24, 2025
@sandhose sandhose merged commit 7d17f34 into main Apr 24, 2025
30 checks passed
@sandhose sandhose deleted the quenting/perf/useragent branch April 24, 2025 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants