From 68de818b868cf946939ff7eee8057fdf6ca75d88 Mon Sep 17 00:00:00 2001 From: Emelia Smith Date: Mon, 13 Oct 2025 14:20:57 +0200 Subject: [PATCH] Merge commit from fork --- streaming/index.js | 26 ++++++++------------------ 1 file changed, 8 insertions(+), 18 deletions(-) diff --git a/streaming/index.js b/streaming/index.js index 3fc5b3613..8ec3188ec 100644 --- a/streaming/index.js +++ b/streaming/index.js @@ -152,17 +152,6 @@ const redisConfigFromEnv = (env) => { }; }; -const PUBLIC_CHANNELS = [ - 'public', - 'public:media', - 'public:local', - 'public:local:media', - 'public:remote', - 'public:remote:media', - 'hashtag', - 'hashtag:local', -]; - // Used for priming the counters/gauges for the various metrics that are // per-channel const CHANNEL_NAMES = [ @@ -171,7 +160,14 @@ const CHANNEL_NAMES = [ 'user:notification', 'list', 'direct', - ...PUBLIC_CHANNELS + 'public', + 'public:media', + 'public:local', + 'public:local:media', + 'public:remote', + 'public:remote:media', + 'hashtag', + 'hashtag:local' ]; const startServer = async () => { @@ -548,12 +544,6 @@ const startServer = async () => { const checkScopes = (req, channelName) => new Promise((resolve, reject) => { log.silly(req.requestId, `Checking OAuth scopes for ${channelName}`); - // When accessing public channels, no scopes are needed - if (PUBLIC_CHANNELS.includes(channelName)) { - resolve(); - return; - } - // The `read` scope has the highest priority, if the token has it // then it can access all streams const requiredScopes = ['read'];