fix: max value of per page

xponential-asia · Oct 26, 2022 · aa699fa · aa699fa
1 parent 973a108
commit aa699fa
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/packages/medusa-plugin-sentry/src/services/sentry.ts b/packages/medusa-plugin-sentry/src/services/sentry.ts
@@ -65,11 +65,11 @@ export default class SentryService extends TransactionBaseService {
 		perPage?: string | number;
 		cursor?: string;
 	}): Promise<SentryFetchResult> {
-		perPage = perPage ?? 100;
+		perPage = Math.min(Number(perPage ?? 100), 100);
 
 		const queryParams = {
 			field: ['transaction', 'tpm()', 'p50()', 'p75()', 'p95()', 'failure_rate', 'apdex()'],
-			per_page: Number(perPage),
+			per_page: perPage,
 			project,
 			query: `event.type:transaction${query ? ' AND ' + query : ''}`,
 			statsPeriod,
@@ -118,11 +118,11 @@ export default class SentryService extends TransactionBaseService {
 		perPage?: string | number;
 		cursor?: string;
 	}): Promise<SentryFetchResult> {
-		perPage = perPage ?? 100;
+		perPage = Math.min(Number(perPage ?? 100), 100);
 
 		const queryParams = {
 			field: ['id', 'transaction.duration', 'timestamp', 'spans.db', 'project'],
-			per_page: Number(perPage),
+			per_page: perPage,
 			project,
 			query: `event.type:transaction AND transaction:"${transaction}"${query ? ' AND ' + query : ''}`,
 			statsPeriod,