fix(core): string-type categories only (#13590)

lib/middleware/parameter.js

@@ -136,6 +136,13 @@ module.exports = async (ctx, next) => {
                     });
                 }
             }
+
+            // handle category
+            if (item.category) {
+                // convert single string to array, and filter only string type category
+                Array.isArray(item.category) || (item.category = [item.category]);
+                item.category = item.category.filter((e) => typeof e === 'string');
+            }
             return item;
         };
 
@@ -172,8 +179,7 @@ module.exports = async (ctx, next) => {
                     const title = item.title || '';
                     const description = item.description || title;
                     const author = item.author || '';
-                    const categoryArray = Array.isArray(item.category) ? item.category : [item.category];
-                    const category = item.category ? categoryArray : [];
+                    const category = item.category || [];
                     const isFilter =
                         engine === 're2'
                             ? regex.matcher(title).find() || regex.matcher(description).find() || regex.matcher(author).find() || category.some((c) => regex.matcher(c).find())
@@ -189,8 +195,7 @@ module.exports = async (ctx, next) => {
                     const title = item.title || '';
                     const description = item.description || title;
                     const author = item.author || '';
-                    const categoryArray = Array.isArray(item.category) ? item.category : [item.category];
-                    const category = item.category ? categoryArray : [];
+                    const category = item.category || [];
                     let isFilter = true;
 
                     const titleRegex = makeRegex(ctx.query.filter_title);
@@ -216,8 +221,7 @@ module.exports = async (ctx, next) => {
                     const title = item.title;
                     const description = item.description || title;
                     const author = item.author || '';
-                    const categoryArray = Array.isArray(item.category) ? item.category : [item.category];
-                    const category = item.category ? categoryArray : [];
+                    const category = item.category || [];
                     let isFilter = true;
 
                     const titleRegex = makeRegex(ctx.query.filterout_title);

lib/v2/test/index.js

@@ -42,6 +42,15 @@ module.exports = async (ctx) => {
                 category: 'Category3',
             },
         ];
+    } else if (ctx.params.id === 'filter-illegal-category') {
+        item.push({
+            title: 'TitleIllegal',
+            description: 'DescriptionIllegal',
+            pubDate: new Date(`2019-3-1`).toUTCString(),
+            link: `https://github.com/DIYgod/RSSHub/issues/1`,
+            author: `DIYgod0`,
+            category: [1, 'CategoryIllegal', true, null, undefined, { type: 'object' }],
+        });
     } else if (ctx.params.id === 'long') {
         item.push({
             title: `Long Title `.repeat(50),

test/middleware/parameter.js

@@ -112,6 +112,14 @@ describe('filter', () => {
         expect(parsed.items[0].title).toBe('Filter Title3');
     });
 
+    it(`filter_category illegal_category`, async () => {
+        const response = await request.get('/test/filter-illegal-category?filter_category=CategoryIllegal');
+        const parsed = await parser.parseString(response.text);
+        expect(parsed.items.length).toBe(1);
+        expect(parsed.items[0].categories.length).toBe(1);
+        expect(parsed.items[0].categories[0]).toBe('CategoryIllegal');
+    });
+
     it(`filter_time`, async () => {
         const response = await request.get('/test/current_time?filter_time=25');
         const parsed = await parser.parseString(response.text);