Merge pull request #21 from ItinerisLtd/manifest-status-header

codepuncher · web-flow · commit 4ab42aaf3c86 · 2025-10-03T16:09:12.000+01:00
fix(manifest): return HTTP 200 not 201
diff --git a/src/AcornFavicons.php b/src/AcornFavicons.php
@@ -297,7 +297,7 @@ public function serveSiteManifest(): void
             return;
         }
 
-        status_header(201);
+        status_header(200);
         header('Content-Type: application/manifest+json; charset=utf-8');
 
         echo wp_json_encode([

Original file line number	Diff line number	Diff line change
`@@ -297,7 +297,7 @@ public function serveSiteManifest(): void`
`297`	`297`	`return;`
`298`	`298`	`}`
`299`	`299`
`300`		`- status_header(201);`
	`300`	`+ status_header(200);`
`301`	`301`	`header('Content-Type: application/manifest+json; charset=utf-8');`
`302`	`302`
`303`	`303`	`echo wp_json_encode([`