Add caching and improve speed #1

Merged

lemonyte merged 5 commits from main into main

2022-10-15 14:59:58 +02:00

lemonyte commented

2022-10-15 06:42:51 +02:00

(Migrated from github.com)

Overview

Added a Cache-Control header, and improved speed by ~60% by switching from StreamingResponse to Response. Detailed test results below.

Also changed:

Added handling for non-existent files, raising 404
Added .gitignore
Removed some unnecessary code

Speed test

Setup:

import requests

original = "https://2hfkxf.deta.dev/space.png"
modified = "https://zn1pu6.deta.dev/space.png"

o_times = []
m_times = []

for i in range(10):
    o = requests.get(original)
    m = requests.get(modified)
    o_times.append(o.elapsed.total_seconds())
    m_times.append(m.elapsed.total_seconds())

print(f"Original: {sum(o_times) / len(o_times)}")
print(f"Modified: {sum(m_times) / len(m_times)}")

Result:

Original: 1.3354535
Modified: 0.4419747

## Overview Added a `Cache-Control` header, and improved speed by ~60% by switching from `StreamingResponse` to `Response`. Detailed test results below. Also changed: - Added handling for non-existent files, raising `404` - Added `.gitignore` - Removed some unnecessary code ## Speed test Setup: ```python import requests original = "https://2hfkxf.deta.dev/space.png" modified = "https://zn1pu6.deta.dev/space.png" o_times = [] m_times = [] for i in range(10): o = requests.get(original) m = requests.get(modified) o_times.append(o.elapsed.total_seconds()) m_times.append(m.elapsed.total_seconds()) print(f"Original: {sum(o_times) / len(o_times)}") print(f"Modified: {sum(m_times) / len(m_times)}") ``` Result: ``` Original: 1.3354535 Modified: 0.4419747 ```

Commenting is not possible because the repository is archived.

No reviewers