Commit b30934e
authored
There was an edge case where, after setting and un-setting the
`powershell.cwd` property in the VS Code client, instead of being `null`
the setting would be the empty string `""`. The client did not care
about this, and so left the server to silently crash on startup. We
previously assumed that by the time the server started, the directory
would be validated by the client, but this was wrong. So now we check
its existence first.
1 parent 4e4fdc3 commit b30934e
File tree
2 files changed
+20
-3
lines changed- src/PowerShellEditorServices/Services/PowerShell/Host
- test/PowerShellEditorServices.Test/Session
2 files changed
+20
-3
lines changedLines changed: 5 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
475 | 475 | | |
476 | 476 | | |
477 | 477 | | |
478 | | - | |
479 | | - | |
480 | | - | |
| 478 | + | |
| 479 | + | |
| 480 | + | |
| 481 | + | |
| 482 | + | |
481 | 483 | | |
482 | 484 | | |
483 | 485 | | |
| |||
Lines changed: 15 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
186 | 186 | | |
187 | 187 | | |
188 | 188 | | |
| 189 | + | |
| 190 | + | |
| 191 | + | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
| 198 | + | |
| 199 | + | |
| 200 | + | |
| 201 | + | |
| 202 | + | |
| 203 | + | |
189 | 204 | | |
190 | 205 | | |
0 commit comments