michael/astro
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fix: Vercel serverless adapter logs too much (#8581)

Browse source
This commit is contained in:
Rishi Raj Jain 2023-09-18 16:52:55 +05:30 committed by GitHub
parent 0586e20e83
commit d0e513f214
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 18 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
.changeset/fluffy-dodos-rule.md Normal file
View file

@ -0,0 +1,5 @@
---
'@astrojs/vercel': patch
---
log only once in serverless adapter

15
packages/integrations/vercel/src/serverless/adapter.ts
View file

@ -245,6 +245,8 @@ You can set functionPerRoute: false to prevent surpassing the limit.`
const filesToInclude = includeFiles?.map((file) => new URL(file, _config.root)) || []; const filesToInclude = includeFiles?.map((file) => new URL(file, _config.root)) || [];
filesToInclude.push(...extraFilesToInclude); filesToInclude.push(...extraFilesToInclude);
validateRuntime();
// Multiple entrypoint support // Multiple entrypoint support
if (_entryPoints.size) { if (_entryPoints.size) {
const getRouteFuncName = (route: RouteData) => route.component.replace('src/pages/', ''); const getRouteFuncName = (route: RouteData) => route.component.replace('src/pages/', '');
@ -312,7 +314,7 @@ You can set functionPerRoute: false to prevent surpassing the limit.`
}; };
} }
function getRuntime() { function validateRuntime() {
const version = process.version.slice(1); // 'v16.5.0' --> '16.5.0' const version = process.version.slice(1); // 'v16.5.0' --> '16.5.0'
const major = version.split('.')[0]; // '16.5.0' --> '16' const major = version.split('.')[0]; // '16.5.0' --> '16'
const support = SUPPORTED_NODE_VERSIONS[major]; const support = SUPPORTED_NODE_VERSIONS[major];
@ -322,7 +324,7 @@ function getRuntime() {
); );
console.warn(`[${PACKAGE_NAME}] Your project will use Node.js 18 as the runtime instead.`); console.warn(`[${PACKAGE_NAME}] Your project will use Node.js 18 as the runtime instead.`);
console.warn(`[${PACKAGE_NAME}] Consider switching your local version to 18.`); console.warn(`[${PACKAGE_NAME}] Consider switching your local version to 18.`);
return 'nodejs18.x'; return;
} }
if (support.status === 'deprecated') { if (support.status === 'deprecated') {
console.warn( console.warn(
@ -336,5 +338,14 @@ function getRuntime() {
); );
console.warn(`[${PACKAGE_NAME}] Consider upgrading your local version to 18.`); console.warn(`[${PACKAGE_NAME}] Consider upgrading your local version to 18.`);
} }
}
function getRuntime() {
const version = process.version.slice(1); // 'v16.5.0' --> '16.5.0'
const major = version.split('.')[0]; // '16.5.0' --> '16'
const support = SUPPORTED_NODE_VERSIONS[major];
if (support === undefined) {
return 'nodejs18.x';
}
return `nodejs${major}.x`; return `nodejs${major}.x`;
} }