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.

Sign up for GitHub

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

Jump to bottom

Replace logo selection route with per-request global variable #5874

Merged
merged 1 commit into from Mar 25, 2021
Merged

Replace logo selection route with per-request global variable #5874

merged 1 commit into from Mar 25, 2021

Commits on Mar 25, 2021

  1. Replace logo selection route with per-request global variable 

    Getting the logo URL via the /org-logo route's redirect meant that the
logo had to be retrieved with every page load, which of course over
Tor can take several seconds. Selecting the logo at the start of each
request and storing its URL in the Flask globals allows the static URL
to be used in the templates, with the default caching of static
assets, eliminating the extra requests.
    @rmol @kushaldas
    rmol authored and kushaldas committed Mar 25, 2021
    Copy the full SHA
    59edd19 View commit details
    Browse the repository at this point in the history