@ixiam/n8n-nodes-civicrm 3.2.0 → 3.2.2

package/dist/src/nodes/transport/GenericFunctions.js

@@ -116,7 +116,6 @@ async function civicrmApiRequest(method, path, body, runtimeBearerToken) {
                 useJwt = true;
             }
             else {
-                console.warn('[CiviCRM] JWT generation returned no token, falling back to API key');
                 this.addExecutionHints({
                     message: 'JWT authentication could not be obtained (no token returned by CiviCRM). Falling back to API Key authentication.',
                     type: 'warning',
@@ -126,7 +125,6 @@ async function civicrmApiRequest(method, path, body, runtimeBearerToken) {
         }
         catch (error) {
             const errorMsg = getHttpErrorDetails(error);
-            console.warn(`[CiviCRM] JWT generation failed: ${errorMsg}. Falling back to API key.`);
             this.addExecutionHints({
                 message: `JWT authentication failed (${errorMsg}). Falling back to API Key authentication.`,
                 type: 'warning',
@@ -154,7 +152,6 @@ async function civicrmApiRequest(method, path, body, runtimeBearerToken) {
                 return response;
             }
             else {
-                console.warn('[CiviCRM] JWT returned empty response. Retrying with API key (JWT may have limited permissions).');
                 this.addExecutionHints({
                     message: 'The JWT-authenticated request returned no data (JWT may have limited permissions). Retrying with API Key authentication.',
                     type: 'warning',
@@ -165,7 +162,6 @@ async function civicrmApiRequest(method, path, body, runtimeBearerToken) {
         }
         catch (error) {
             const errorMsg = getHttpErrorDetails(error);
-            console.warn('[CiviCRM] JWT request failed. Retrying with API key.');
             this.addExecutionHints({
                 message: `The JWT-authenticated request failed (${errorMsg}). Retrying with API Key authentication.`,
                 type: 'warning',
@@ -186,11 +182,7 @@ async function civicrmApiRequest(method, path, body, runtimeBearerToken) {
         json: true,
     };
     try {
-        const response = await this.helpers.httpRequest.call(this, apiKeyOptions);
-        if (useJwt && jwtToken) {
-            console.log('[CiviCRM] API key request successful (JWT was insufficient, using API key as fallback)');
-        }
-        return response;
+        return await this.helpers.httpRequest.call(this, apiKeyOptions);
     }
     catch (error) {
         throw new n8n_workflow_1.NodeApiError(this.getNode(), error);

package/dist/src/nodes/transport/JwtAuth.js

@@ -8,28 +8,6 @@ const contactIdCache = {}; // Cache resolved contact IDs
 function isJwtAuthEnabled(credentials) {
     return (credentials === null || credentials === void 0 ? void 0 : credentials.enableJwtAuth) === true;
 }
-/**
- * Extracts a readable message from an HTTP error, including the response body
- * (e.g. CiviCRM's "Login not permitted. Must satisfy guard (perm, site_key)."),
- * which the bare `error.message` (e.g. "Request failed with status code 401")
- * does not include.
- */
-function getErrorDetails(error) {
-    var _a, _b, _c;
-    const errorObj = error;
-    const status = (_a = errorObj === null || errorObj === void 0 ? void 0 : errorObj.response) === null || _a === void 0 ? void 0 : _a.status;
-    const data = (_b = errorObj === null || errorObj === void 0 ? void 0 : errorObj.response) === null || _b === void 0 ? void 0 : _b.data;
-    const message = (_c = errorObj === null || errorObj === void 0 ? void 0 : errorObj.message) !== null && _c !== void 0 ? _c : String(error);
-    const body = typeof data === 'string' && data.trim()
-        ? data
-        : data && typeof data === 'object'
-            ? JSON.stringify(data)
-            : undefined;
-    if (body) {
-        return status ? `${message} | HTTP ${status} body: ${body}` : `${message} | body: ${body}`;
-    }
-    return status ? `${message} | HTTP ${status}` : message;
-}
 /**
  * Resolve the current authenticated user's contact ID from Contact/get.
  * Finds the contact that owns the given api_key.
@@ -60,16 +38,13 @@ async function resolveContactId(context, baseUrl, apiToken) {
         });
         const contacts = (response === null || response === void 0 ? void 0 : response.values) || [];
         if (contacts.length === 0) {
-            console.warn('[CiviCRM] Contact/get found no contact with this api_key - JWT auto-resolve will be skipped');
             return undefined;
         }
         const contactId = contacts[0].id;
         contactIdCache[cacheKey] = contactId;
         return contactId;
     }
-    catch (error) {
-        const errorMsg = getErrorDetails(error);
-        console.warn(`[CiviCRM] Failed to auto-resolve contact ID: ${errorMsg}. Will attempt API key fallback.`);
+    catch {
         return undefined;
     }
 }
@@ -116,26 +91,15 @@ async function getServerIssuedJwt(context, baseUrl, apiToken, contactId = 0, ttl
         });
         const cred = ((_b = (_a = response === null || response === void 0 ? void 0 : response.values) === null || _a === void 0 ? void 0 : _a[0]) === null || _b === void 0 ? void 0 : _b.cred) || '';
         if (!cred.startsWith('Bearer ')) {
-            console.warn('[CiviCRM] Invalid JWT response: missing Bearer token');
             return undefined;
         }
         const token = cred.substring(7); // Remove "Bearer " prefix
         serverIssuedCache[cacheKey] = { token, expiresAt: now + ttl * 1000 };
         return token;
     }
-    catch (error) {
-        const errorMsg = getErrorDetails(error);
-        const isPermissionDenied = errorMsg.includes('Authorization failed') ||
-            errorMsg.includes('403') ||
-            errorMsg.includes('Permission denied') ||
-            errorMsg.includes('Login not permitted') ||
-            errorMsg.includes('Must satisfy guard');
-        if (isPermissionDenied) {
-            // User lacks permissions for JWT. Fall back to API key auth.
-            console.warn(`[CiviCRM JWT] User lacks permissions for AuthxCredential/create (${errorMsg}). Will use API key authentication instead.`);
-            return undefined;
-        }
-        console.warn(`[CiviCRM JWT] Failed to obtain JWT: ${errorMsg}. Falling back to API key.`);
+    catch {
+        // Any failure here (permission denied, network error, malformed response) falls
+        // back to API key auth in the caller -- see GenericFunctions.civicrmApiRequest.
         return undefined;
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ixiam/n8n-nodes-civicrm",
-  "version": "3.2.0",
+  "version": "3.2.2",
   "description": "Full-featured CiviCRM API v4 integration for n8n",
   "license": "MIT",
   "keywords": [
@@ -52,7 +52,6 @@
   "devDependencies": {
     "@n8n/node-cli": "^0.23.1",
     "@types/jest": "^30.0.0",
-    "@types/jsonwebtoken": "^9.0.10",
     "@types/node": "^20.11.30",
     "eslint": "9.32.0",
     "jest": "^30.5.1",
@@ -61,9 +60,7 @@
     "ts-jest": "^29.4.12",
     "typescript": "5.9.2"
   },
-  "dependencies": {
-    "jsonwebtoken": "^9.0.3"
-  },
+  "dependencies": {},
   "peerDependencies": {
     "n8n-workflow": "*"
   },