Add caching headers to existing Kolibri public APIs

[bjester]

Background

We're adding new V2 APIs from Kolibri to Studio, and we're adding Cloudflare page rules to make sure that we can obtain optimal caching of the API responses

Desired behavior

The existing /api/public endpoints:

• should return an appropriate Last-modified and support handling the If-modified-since header
• should return cache control headers: Cache-control: public, stale-while-revalidate=100, max-age=300

Current behavior

These endpoints do not return or support caching

Value add

Better edge caching through Cloudflare, meaning better worldwide browsing of public channels through Kolibri.

[alphaket14]

Hi, @bjester . I'm new to studio and would like to start my journey with this issue. I understand this issue but don't know much about edge caching. So can you please guide me to know about them and help to find the right files which need to be changed under this issue? Thanks :)

[bjester]

Hi @alphaket14, thank you for your interest. Here are some details:

• First, this PR needs to be merged before you can work on this, because it moves the V1 APIs to a new location
• Use the kolibri-public branch as base and target for your PR
• There are decorators in the locale and session middleware that will prevent adding headers (like cookies) that will break caching
• Update the API endpoints in contentcuration/kolibri_public/views_v1 to respond with the appropriate headers
• Use Channel.last_published datetime for Last-modified header

[bjester]

@alphaket14 Apologies, I forgot one thing. There is a caching decorator that should exist in the kolibri-public branch which you can look at for similar functionality

studio/contentcuration/kolibri_public/views.py

Line 54 in 5e13169

def metadata_cache(some_func):

[rtibbles]

One other thing to note is this comment about how we handle caching for as yet unrecognized device versions in the Info endpoint - we should follow the recommendation of @bjester here: #3987 (comment) and disable the caching headers conditionally when we have an unrecognized device info version.

[bjester]

Hi @alphaket14, just checking in that you're indeed intending to address this?

[alphaket14]

Hi @bjester , I was facing issue during the wsl local setup of /studio branch,
The issue was this : [(https://stackoverflow.com/questions/44678725/cannot-connect-to-the-docker-daemon-at-unix-var-run-docker-sock-is-the-docker)
]
Although I had docker desktop running,
This occurred when I tried running the command : make run-services

Then I dual booted my PC with ubuntu 22.04, currently setting up the local development
Will contact you very soon, and yes I am intending to solve this problem

[alphaket14]

@bjester I am not able to resolve this issue, can i get some help
This occured when I made the
make run-services

This time docker daemon was running

Also the error mentioned module celery is also present