Merge pull request #393 from gjsjohnmurray/fix-392

fix #392 no longer require %SYS namespace access for 'Add Server Namespace to Workspace' command

Author: gjsjohnmurray

src/api/index.ts

@@ -340,12 +340,16 @@ export class AtelierAPI {
     }
   }
 
+  /**
+   * Use the serverInfo endpoint. Connection object does not need to have specified a namespace when instantiated.
+   */
   public serverInfo(): Promise<Atelier.Response<Atelier.Content<Atelier.ServerInfo>>> {
     return this.request(0, "GET").then((info) => {
       if (info && info.result && info.result.content && info.result.content.api > 0) {
         const data = info.result.content;
         const apiVersion = data.api;
-        if (!data.namespaces.includes(this.ns.toUpperCase())) {
+        // If this is a namespace-specific connection, throw if it doesn't exist or if the user has no access to it
+        if (this.ns && !data.namespaces.includes(this.ns.toUpperCase())) {
           throw {
             code: "WrongNamespace",
             message: `This server does not have specified namespace '${this.ns}'.\n

src/commands/addServerNamespaceToWorkspace.ts

@@ -24,7 +24,7 @@ export async function addServerNamespaceToWorkspace(): Promise<void> {
     return;
   }
   // Get its namespace list
-  let uri = vscode.Uri.parse(`isfs://${serverName}/?ns=%SYS`);
+  let uri = vscode.Uri.parse(`isfs://${serverName}/`);
   await resolveConnectionSpec(serverName);
   // Prepare a displayable form of its connection spec as a hint to the user
   const connSpec = await serverManagerApi.getServerSpec(serverName);