Merge pull request #650 from bash-lsp/check-config-for-null

skovhus · web-flow · commit 5a85c10eb279 · 2023-01-02T20:22:29.000+01:00
Validate that the config object is not null
diff --git a/server/src/server.ts b/server/src/server.ts
@@ -204,7 +204,7 @@ export default class BashServer {
   }
 
   private updateConfiguration(configObject: any): boolean {
-    if (typeof configObject === 'object') {
+    if (typeof configObject === 'object' && configObject !== null) {
       try {
         const newConfig = config.ConfigSchema.parse(configObject)
 

Original file line number	Diff line number	Diff line change
`@@ -204,7 +204,7 @@ export default class BashServer {`
`204`	`204`	`}`
`205`	`205`
`206`	`206`	`private updateConfiguration(configObject: any): boolean {`
`207`		`- if (typeof configObject === 'object') {`
	`207`	`+ if (typeof configObject === 'object' && configObject !== null) {`
`208`	`208`	`try {`
`209`	`209`	`const newConfig = config.ConfigSchema.parse(configObject)`
`210`	`210`