feat: send `Vary: Accept` on all relevant responses
Created by: dakkar
Summary
We have a bunch of URIs that can return HTML or application/activity+json
(look at packages/backend/src/server/ActivityPubServerService.ts
)
Some of those (/notes/:id
and /notes/:id/activity
) add a Vary: Accept
to their AP responses, but nothing else does. This may confuse browsers and caching proxies.
something like vary(reply.raw, 'Accept')
(with import vary from 'vary'
) in the right places should be enough.
Purpose
prevent this https://discord.com/channels/1126590789322035230/1163467816284991591/1197642031586082897
Do you want to implement this feature yourself?
-
Yes, I will implement this by myself and send a pull request