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

gzip and cache track maps #407

Merged
merged 15 commits into from
Jan 16, 2025
Merged

gzip and cache track maps #407

merged 15 commits into from
Jan 16, 2025

Conversation

merlinorg
Copy link
Contributor

@merlinorg merlinorg commented Jan 14, 2025

Track maps are big and slow. Compressing and caching makes the site much zippier.

Resolves #334.

freezing/web/views/api.py Fixed Show resolved Hide resolved
freezing/web/views/api.py Fixed Show fixed Hide fixed
freezing/web/views/api.py Fixed Show fixed Hide fixed
freezing/web/views/api.py Fixed Show fixed Hide fixed
freezing/web/views/api.py Fixed Show fixed Hide fixed
@merlinorg merlinorg force-pushed the gzip-cache-track-maps branch from ebcf177 to d01ad8e Compare January 14, 2025 18:40
…sion


Make CodeQL happy.

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
freezing/web/views/api.py Fixed Show fixed Hide fixed
freezing/web/views/api.py Fixed Show fixed Hide fixed
example.cfg Outdated Show resolved Hide resolved
Copy link
Member

@obscurerichard obscurerichard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like it will work great but let's adjust per the comments a bit.

Copy link
Member

@obscurerichard obscurerichard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻 :shipit:

The docker-compose.yml setting now works for the new cache dir.

This emits a 500 Server Error if something goes wrong now.
freezing/web/views/api.py Fixed Show fixed Hide fixed
freezing/web/views/api.py Fixed Show fixed Hide fixed
freezing/web/views/api.py Fixed Show fixed Hide fixed
freezing/web/views/api.py Fixed Show fixed Hide fixed
freezing/web/views/api.py Fixed Show fixed Hide fixed
freezing/web/views/api.py Fixed Show fixed Hide fixed
obscurerichard and others added 2 commits January 16, 2025 10:32
…in path expression

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
freezing/web/views/api.py Fixed Show fixed Hide fixed
…in path expression

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
freezing/web/views/api.py Fixed Show fixed Hide fixed
…in path expression

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@obscurerichard
Copy link
Member

I'm going to do a little experiment here and let GitHub Copilot autofix progressively nip the potential path manipulation problems in the bud, bit by bit... you can only do one at a time which is annoying because there are about 5 fixes it is suggesting.

freezing/web/views/api.py Fixed Show fixed Hide fixed
freezing/web/views/api.py Fixed Show fixed Hide fixed
obscurerichard and others added 2 commits January 16, 2025 11:43
…in path expression

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@obscurerichard obscurerichard merged commit c869ffc into main Jan 16, 2025
7 checks passed
@obscurerichard obscurerichard deleted the gzip-cache-track-maps branch January 16, 2025 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Cache map json
2 participants