Merge pull request #247 from nitrictech/fix/catch-all-files-entrypoints-sites

tjholm · web-flow · commit e0914e3d9d23 · 2021-11-29T11:34:27.000+11:00
fix: ensure all site routes get sent to index
diff --git a/packages/base/src/tasks/run/entrypoints.test.ts b/packages/base/src/tasks/run/entrypoints.test.ts
@@ -77,6 +77,7 @@ describe('createNginxConfig', () => {
 			expect(config).toContain(`
 				location / {
 					root /www/test;
+					try_files $uri $uri/ /index.html;
 				}
 			`);
 		});
diff --git a/packages/base/src/tasks/run/entrypoints.ts b/packages/base/src/tasks/run/entrypoints.ts
@@ -76,6 +76,7 @@ export function createNginxConfig(entrypoint: NamedObject<NitricEntrypoint>, sta
 					(s) => `
 				location ${s.path} {
 					root /www/${s.target};
+					try_files $uri $uri/ /index.html;
 				}
 			`,
 				)

Original file line number	Diff line number	Diff line change
`@@ -77,6 +77,7 @@ describe('createNginxConfig', () => {`
`77`	`77`	expect(config).toContain(`
`78`	`78`	`location / {`
`79`	`79`	`root /www/test;`
	`80`	`+ try_files $uri $uri/ /index.html;`
`80`	`81`	`}`
`81`	`82`	`);
`82`	`83`	`});`
Original file line number	Diff line number	Diff line change
`@@ -76,6 +76,7 @@ export function createNginxConfig(entrypoint: NamedObject<NitricEntrypoint>, sta`
`76`	`76`	(s) => `
`77`	`77`	`location ${s.path} {`
`78`	`78`	`root /www/${s.target};`
	`79`	`+ try_files $uri $uri/ /index.html;`
`79`	`80`	`}`
`80`	`81`	`,
`81`	`82`	`)`