Skip to content

Commit e771c6b

Browse files
committed
Return 500 status when asset upload error page is rendered
1 parent fca9c19 commit e771c6b

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

.changeset/bright-impalas-float.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@
22
'@shopify/theme': minor
33
---
44

5-
Render error overlay when `theme dev` encounters asset upload errors
5+
Render error overlay when `theme dev` encounters asset upload errors. A 500 status code is returned when the error overlay is rendered.

packages/theme/src/cli/utilities/theme-environment/html.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,8 @@ export function getHtmlHandler(theme: Theme, ctx: DevServerContext) {
3333
assertThemeId(response, html, String(theme.id))
3434

3535
if (ctx.options.errorOverlay !== 'silent' && ctx.localThemeFileSystem.uploadErrors.size > 0) {
36+
setResponseStatus(event, 500, 'Failed to Upload Theme Files')
37+
setResponseHeader(event, 'Content-Type', 'text/html')
3638
html = getErrorPage({
3739
title: 'Failed to Upload Theme Files',
3840
header: 'Upload Errors',

0 commit comments

Comments
 (0)