fix: better error messages for updates

fontsource · Apr 30, 2023 · df10b10 · df10b10
1 parent 4611ad9
commit df10b10
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/.github/workflows/website.yml b/.github/workflows/website.yml
@@ -28,4 +28,4 @@ jobs:
           bearerToken: ${{ secrets.WESBITE_UPDATE_TOKEN }}
           data: '{"fonts": true, "algolia": true, "download": true, "axisRegistry": true}'
           retry: 1
-          retryDelay: 30000
+          retryWait: 15000
diff --git a/website/app/routes/actions/update.ts b/website/app/routes/actions/update.ts
@@ -26,13 +26,12 @@ export const loader: LoaderFunction = async () => {
 export const action: ActionFunction = async ({ request }) => {
 	const data: UpdateData = await request.json();
 	const header = await request.headers.get('Authorization');
-	invariant(header, 'No authorization header was sent with the request');
-
-	invariant(data, 'No data was sent with the request');
-
-	if (header !== `Bearer ${process.env.UPDATE_TOKEN}`) {
+	if (!header || header !== `Bearer ${process.env.UPDATE_TOKEN}`) {
 		return new Response('Invalid update bearer token', { status: 401 });
 	}
+	if (!data) {
+		return new Response('Invalid update data', { status: 400 });
+	}
 
 	if (data.fonts) {
 		console.log('Updating fonts');