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

Protect against shimmed process usage in Vercel Edge (#5969)

Browse source 
* Protect against shimmed process usage in Vercel Edge

* Adding a changeset
This commit is contained in:
Matthew Phillips 2023-01-24 21:11:09 -05:00
parent 21a3774d53
commit f3130d63a2
2 changed files with 19 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
.changeset/nasty-dolphins-cry.md Normal file
View file

@ -0,0 +1,7 @@
---
'astro': patch
---
Fix usage of logger in Vercel Edge
This protects against usage of `process` global in shimmed environments.

16
packages/astro/src/core/logger/core.ts
View file

@ -101,10 +101,18 @@ function padStr(str: string, len: number) {
export let defaultLogLevel: LoggerLevel; export let defaultLogLevel: LoggerLevel;
if (typeof process !== 'undefined') { if (typeof process !== 'undefined') {
if (process.argv.includes('--verbose')) { // This could be a shimmed environment so we don't know that `process` is the full
defaultLogLevel = 'debug'; // NodeJS.process. This code treats it as a plain object so TS doesn't let us
} else if (process.argv.includes('--silent')) { // get away with incorrect assumptions.
defaultLogLevel = 'silent'; let proc: object = process;
if('argv' in proc && Array.isArray(proc.argv)) {
if (proc.argv.includes('--verbose')) {
defaultLogLevel = 'debug';
} else if (proc.argv.includes('--silent')) {
defaultLogLevel = 'silent';
} else {
defaultLogLevel = 'info';
}
} else { } else {
defaultLogLevel = 'info'; defaultLogLevel = 'info';
} }