n8n-nodes-qlik-cloud 1.0.17 → 1.1.0

package/dist/nodes/QlikCloud/resources/assistants/index.js

@@ -3,6 +3,33 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.assistantDescription = exports.assistantsInvokeThreadDescription = exports.assistantsStreamThreadDescription = exports.assistantsCreateThreadDescription = exports.assistantsListThreadsDescription = exports.assistantsCreateStarterDescription = exports.assistantsListStartersDescription = exports.assistantsDeleteDescription = exports.assistantsBulkSearchSourcesDescription = exports.assistantsGetFeedbackDescription = exports.assistantsSearchDescription = exports.assistantsUpdateDescription = exports.assistantsCreateDescription = exports.assistantsGetDescription = exports.assistantsGetAllDescription = void 0;
 // Get All Assistants
 exports.assistantsGetAllDescription = [
+    {
+        displayName: 'Return All',
+        name: 'returnAll',
+        type: 'boolean',
+        default: false,
+        displayOptions: {
+            show: {
+                resource: ['assistants'],
+                operation: ['getAll'],
+            },
+        },
+        description: 'Whether to return all assistants',
+    },
+    {
+        displayName: 'Limit',
+        name: 'limit',
+        type: 'number',
+        default: 50,
+        displayOptions: {
+            show: {
+                resource: ['assistants'],
+                operation: ['getAll'],
+                returnAll: [false],
+            },
+        },
+        description: 'The number of assistants to get',
+    },
     {
         displayName: 'Options',
         name: 'options',
@@ -16,13 +43,6 @@ exports.assistantsGetAllDescription = [
             },
         },
         options: [
-            {
-                displayName: 'Limit',
-                name: 'limit',
-                type: 'number',
-                default: 50,
-                description: 'The number of assistants to get',
-            },
             {
                 displayName: 'Next Page',
                 name: 'next',

package/dist/nodes/QlikCloud/shared/transport.d.ts

@@ -1,2 +1,2 @@
-import type { IExecuteFunctions, IHttpRequestMethods } from 'n8n-workflow';
-export declare function qlikApiRequest(this: IExecuteFunctions, method: IHttpRequestMethods, endpoint: string, body?: Record<string, any>, qs?: Record<string, any>): Promise<Record<string, any>>;
+import type { IExecuteFunctions, IHttpRequestMethods, IHttpRequestOptions } from 'n8n-workflow';
+export declare function qlikApiRequest(this: IExecuteFunctions, method: IHttpRequestMethods, endpoint: string, body?: any, qs?: Record<string, any>, returnFullResponse?: boolean, extraOptions?: Partial<IHttpRequestOptions>): Promise<any>;

package/dist/nodes/QlikCloud/shared/transport.js

@@ -2,23 +2,29 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.qlikApiRequest = qlikApiRequest;
 const n8n_workflow_1 = require("n8n-workflow");
-async function qlikApiRequest(method, endpoint, body, qs) {
+async function qlikApiRequest(method, endpoint, body, qs, returnFullResponse = false, extraOptions = {}) {
     const credentials = await this.getCredentials('qlikCloudApi');
     if (!credentials) {
         throw new n8n_workflow_1.NodeApiError(this.getNode(), {
             message: 'No credentials found for Qlik Cloud API',
         });
     }
-    const baseUrl = credentials.baseUrl;
+    const baseUrl = credentials.baseUrl.replace(/\/+$/, '');
     const accessToken = credentials.accessToken;
+    const url = `${baseUrl}${endpoint.startsWith('/') ? endpoint : `/${endpoint}`}`;
+    const headers = {
+        Authorization: `Bearer ${accessToken}`,
+        'Content-Type': 'application/json',
+        ...(extraOptions.headers || {}),
+    };
     const options = {
-        headers: {
-            Authorization: `Bearer ${accessToken}`,
-            'Content-Type': 'application/json',
-        },
         method,
-        url: `${baseUrl}${endpoint}`,
+        url,
         qs,
+        json: extraOptions.json ?? true,
+        returnFullResponse,
+        ...extraOptions,
+        headers,
     };
     if (body !== undefined) {
         options.body = body;
@@ -27,8 +33,9 @@ async function qlikApiRequest(method, endpoint, body, qs) {
         return await this.helpers.httpRequest(options);
     }
     catch (error) {
-        throw new n8n_workflow_1.NodeApiError(this.getNode(), {
-            message: error.message,
+        const errMessage = error instanceof Error ? error.message : typeof error === 'string' ? error : JSON.stringify(error);
+        throw new n8n_workflow_1.NodeApiError(this.getNode(), { message: errMessage }, {
+            message: 'Qlik Cloud API request failed',
         });
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "n8n-nodes-qlik-cloud",
-  "version": "1.0.17",
+  "version": "1.1.0",
   "description": "n8n node for Qlik Cloud REST APIs integration",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -10,8 +10,7 @@
     "dev": "tsc --watch",
     "lint": "eslint src/",
     "test": "jest",
-    "publish": "npm run build && npm version patch && npm publish",
-    "prepublishOnly": "npm run build"
+    "publish-patch": "npm run build && npm version patch && npm publish"
   },
   "n8n": {
     "nodes": [

package/src/credentials/QlikCloudOAuth2Api.credentials.ts

@@ -1,11 +1,6 @@
-import type {
-	ICredentialType,
-	INodeProperties,
-} from 'n8n-workflow';
+import type { ICredentialType, INodeProperties } from 'n8n-workflow';
 
-const scopes = [
-	'https://analysis.windows.net/powerbi/api/.default',
-];
+const DEFAULT_SCOPES = ['openid', 'profile', 'email', 'offline_access'];
 
 export class QlikCloudOAuth2Api implements ICredentialType {
 	name = 'qlikCloudOAuth2Api';
@@ -28,21 +23,19 @@ export class QlikCloudOAuth2Api implements ICredentialType {
 			name: 'authUrl',
 			type: 'hidden',
 			default: '',
-			required: true,
 		},
 		{
 			displayName: 'Access Token URL',
 			name: 'accessTokenUrl',
 			type: 'hidden',
 			default: '',
-			required: true,
 		},
 		{
 			displayName: 'Scope',
 			name: 'scope',
-			type: 'hidden',
-			default: scopes.join(' '),
-			description: 'OAuth2 scopes required for Qlik Cloud API access',
+			type: 'string',
+			default: DEFAULT_SCOPES.join(' '),
+			description: 'OAuth2 scopes requested from Qlik Cloud (space-separated)',
 		},
 		{
 			displayName: 'Authentication',
@@ -54,27 +47,31 @@ export class QlikCloudOAuth2Api implements ICredentialType {
 
 	async preAuthentication(this: any, credentials: any) {
 		const baseUrl = credentials.baseUrl as string;
-		
-		// Extract tenant and region from base URL
-		// Expected format: https://tenant.region.qlikcloud.com
-		const urlParts = new URL(baseUrl);
-		const hostname = urlParts.hostname;
-		const parts = hostname.split('.');
-		
-		if (parts.length < 3) {
+		let parsed: URL;
+
+		try {
+			parsed = new URL(baseUrl);
+		} catch (error) {
 			throw new Error('Invalid Qlik Cloud tenant URL format');
 		}
 
-		const tenant = parts[0];
-		const region = parts[1];
+		if (parsed.protocol !== 'https:') {
+			throw new Error('Qlik Cloud tenant URL must use HTTPS');
+		}
 
-		// Construct OAuth endpoints
-		const authEndpoint = `https://auth.${region}.qlikcloud.com/oauth/authorize`;
-		const tokenEndpoint = `https://auth.${region}.qlikcloud.com/oauth/token`;
+		if (!parsed.hostname.endsWith('.qlikcloud.com')) {
+			throw new Error('Tenant URL must end with .qlikcloud.com');
+		}
+
+		const oauthBase = `${parsed.origin}/oauth`;
+
+		credentials.authUrl = `${oauthBase}/authorize`;
+		credentials.accessTokenUrl = `${oauthBase}/token`;
+
+		if (!credentials.scope) {
+			credentials.scope = DEFAULT_SCOPES.join(' ');
+		}
 
-		credentials.authUrl = authEndpoint;
-		credentials.accessTokenUrl = tokenEndpoint;
-		
 		return credentials;
 	}
 }