FIX: allow rss feeds to have an enclosure with length of 0 (#9967)

madcampos · florian-lefebvre · web-flow · commit 8b8f26fdf2af · 2024-02-04T20:42:26.000+01:00
* fix: allow rss feeds to have an enclosure with length of 0

* chore: add changeset

* fix: typo on test

* fix: update changeset description

Co-authored-by: Florian Lefebvre &lt;contact@florian-lefebvre.dev&gt;

---------

Co-authored-by: Florian Lefebvre &lt;contact@florian-lefebvre.dev&gt;
diff --git a/.changeset/tidy-spoons-suffer.md b/.changeset/tidy-spoons-suffer.md
@@ -0,0 +1,5 @@
+---
+"@astrojs/rss": patch
+---
+
+Allows `enclosure' to have a length of 0
diff --git a/packages/astro-rss/src/schema.ts b/packages/astro-rss/src/schema.ts
@@ -16,7 +16,7 @@ export const rssSchema = z.object({
 	enclosure: z
 		.object({
 			url: z.string(),
-			length: z.number().positive().int().finite(),
+			length: z.number().nonnegative().int().finite(),
 			type: z.string(),
 		})
 		.optional(),
diff --git a/packages/astro-rss/test/rss.test.js b/packages/astro-rss/test/rss.test.js
@@ -227,4 +227,27 @@ describe('getRssString', () => {
 		}
 		chai.expect(error).to.be.null;
 	});
+
+	it('should not fail when an enclosure has a length of 0', async () => {
+		const str = await getRssString({
+			title,
+			description,
+			items: [
+				{
+					title: 'Title',
+					pubDate: new Date().toISOString(),
+					description: 'Description',
+					link: '/link',
+					enclosure: {
+						url: '/enclosure',
+						length: 0,
+						type: 'audio/mpeg',
+					},
+				},
+			],
+			site,
+		});
+
+		chai.expect(str).to.not.throw;
+	});
 });

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +"@astrojs/rss": patch
 +---
++
 +Allows `enclosure' to have a length of 0