fix(aProblem): do not cache by default

BREAKING CHANGE: this changes the behaviour of aProblem

Author: coderbyheart

src/aProblem.spec.ts

@@ -14,7 +14,8 @@ void describe('aProblem()', () => {
 				statusCode: 409,
 				headers: {
 					'content-type': 'application/problem+json',
-					'Cache-Control': 'public, max-age=60',
+					// 'Cache-Control': 'public, max-age=60',
+					'Cache-Control': 'no-store',
 				},
 				body: JSON.stringify({
 					'@context': Context.problemDetail.toString(),
@@ -23,4 +24,20 @@ void describe('aProblem()', () => {
 				}),
 			},
 		))
+
+	void it('can set cache control', () =>
+		assert.partialDeepStrictEqual(
+			aProblem(
+				{
+					title: `A Conflict!`,
+					status: 409,
+				},
+				60,
+			),
+			{
+				headers: {
+					'Cache-Control': 'public, max-age=60',
+				},
+			},
+		))
 })

src/aProblem.ts

@@ -4,12 +4,13 @@ import type { APIGatewayProxyStructuredResultV2 } from 'aws-lambda'
 
 export const aProblem = (
 	problem: Omit<Static<typeof ProblemDetail>, '@context'>,
-	cacheForSeconds: number = 60,
+	cacheForSeconds: number = 0,
 ): APIGatewayProxyStructuredResultV2 => ({
 	statusCode: problem.status,
 	headers: {
 		'content-type': 'application/problem+json',
-		'Cache-Control': `public, max-age=${cacheForSeconds}`,
+		'Cache-Control':
+			cacheForSeconds > 0 ? `public, max-age=${cacheForSeconds}` : 'no-store',
 	},
 	body: JSON.stringify({
 		'@context': Context.problemDetail.toString(),