fix(route/mastodon): Correct the only_media param (DIYgod#17531)

vcshixin · Nov 11, 2024 · 787cb1c · 787cb1c
1 parent a490992
commit 787cb1c
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/lib/routes/mastodon/account-id.ts b/lib/routes/mastodon/account-id.ts
@@ -36,7 +36,7 @@ export const route: Route = {
 async function handler(ctx) {
     const site = ctx.req.param('site');
     const account_id = ctx.req.param('account_id');
-    const only_media = ctx.req.param('only_media') ? 'true' : 'false';
+    const only_media = ctx.req.param('only_media') === 'true' ? 'true' : 'false';
     if (!config.feature.allow_user_supply_unsafe_domain && !utils.allowSiteList.includes(site)) {
         throw new ConfigNotFoundError(`This RSS is disabled unless 'ALLOW_USER_SUPPLY_UNSAFE_DOMAIN' is set to 'true'.`);
     }

diff --git a/lib/routes/mastodon/acct.ts b/lib/routes/mastodon/acct.ts
@@ -35,7 +35,7 @@ However, you can still specify these route-specific configurations if you need t
 
 async function handler(ctx) {
     const acct = ctx.req.param('acct');
-    const only_media = ctx.req.param('only_media') ? 'true' : 'false';
+    const only_media = ctx.req.param('only_media') === 'true' ? 'true' : 'false';
 
     const { site, account_id } = await utils.getAccountIdByAcct(acct);
 

diff --git a/lib/routes/mastodon/timeline-local.ts b/lib/routes/mastodon/timeline-local.ts
@@ -36,7 +36,7 @@ export const route: Route = {
 
 async function handler(ctx) {
     const site = ctx.req.param('site');
-    const only_media = ctx.req.param('only_media') ? 'true' : 'false';
+    const only_media = ctx.req.param('only_media') === 'true' ? 'true' : 'false';
     if (!config.feature.allow_user_supply_unsafe_domain && !utils.allowSiteList.includes(site)) {
         throw new ConfigNotFoundError(`This RSS is disabled unless 'ALLOW_USER_SUPPLY_UNSAFE_DOMAIN' is set to 'true'.`);
     }
@@ -47,7 +47,7 @@ async function handler(ctx) {
     const list = response.data;
 
     return {
-        title: `Local Public${ctx.req.param('only_media') ? ' Media' : ''} Timeline on ${site}`,
+        title: `Local Public${ctx.req.param('only_media') === 'true' ? ' Media' : ''} Timeline on ${site}`,
         link: `https://${site}`,
         item: utils.parseStatuses(list),
     };

diff --git a/lib/routes/mastodon/timeline-remote.ts b/lib/routes/mastodon/timeline-remote.ts
@@ -36,7 +36,7 @@ export const route: Route = {
 
 async function handler(ctx) {
     const site = ctx.req.param('site');
-    const only_media = ctx.req.param('only_media') ? 'true' : 'false';
+    const only_media = ctx.req.param('only_media') === 'true' ? 'true' : 'false';
     if (!config.feature.allow_user_supply_unsafe_domain && !utils.allowSiteList.includes(site)) {
         throw new ConfigNotFoundError(`This RSS is disabled unless 'ALLOW_USER_SUPPLY_UNSAFE_DOMAIN' is set to 'true'.`);
     }
@@ -47,7 +47,7 @@ async function handler(ctx) {
     const list = response.data;
 
     return {
-        title: `Federated Public${ctx.req.param('only_media') ? ' Media' : ''} Timeline on ${site}`,
+        title: `Federated Public${ctx.req.param('only_media') === 'true' ? ' Media' : ''} Timeline on ${site}`,
         link: `https://${site}`,
         item: utils.parseStatuses(list),
     };