zowe / zen

Eclipse Public License 2.0
1 stars 6 forks source link

Check if `zowe.*Directory` is not nested in `zowe.runtimeDirectory` #243

Open Martin-Zeithaml opened 2 months ago

Martin-Zeithaml commented 2 months ago

Schema validation is checking, if the directories used for Zowe are valid USS paths. But that's not sufficient, user can define e.g. zowe.logDirectory (or other directories) "inside" zowe.runtimeDirectory. This is not good for couple reasons:

Such setting should invoke (at least) a warning:

zowe:
  runtimeDirectory: /global/zowe
  logDirectory: /global/zowe/logs
  workspaceDirectory: /global/zowe/workspace
  extensionDirectory: /global/zowe/extensions

There are more directories used, like java, node, certificates, but the check for the directories in the example should be fine.

skurnevich commented 2 months ago

I'v added a basic warning as part of this PR But it could be done in a better way, so lets keep this task as open.