admin/n8n-enterprise-unlocked
1
0
Fork 0
mirror of https://github.com/Abdulazizzn/n8n-enterprise-unlocked.git synced 2025-12-17 18:12:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

⚡ Add OAuth2 authentication to Pipedrive Trigger Node

Browse Source
This commit is contained in:
Jan Oberhauser
2020-11-10 21:27:20 +01:00
parent 86d19fb11f
commit 1630d20e39
1 changed files with 41 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

47
packages/nodes-base/nodes/Pipedrive/PipedriveTrigger.node.ts
View File

@@ -54,13 +54,31 @@ export class PipedriveTrigger implements INodeType {
{
name: 'pipedriveApi',
required: true,
displayOptions: {
show: {
authentication: [
'apiToken',
],
},
},
},
{
name: 'pipedriveOAuth2Api',
required: true,
displayOptions: {
show: {
authentication: [
'oAuth2',
],
},
},
},
{
name: 'httpBasicAuth',
required: true,
displayOptions: {
show: {
authentication: [
incomingAuthentication: [
'basicAuth',
],
},
@@ -80,6 +98,23 @@ export class PipedriveTrigger implements INodeType {
displayName: 'Authentication',
name: 'authentication',
type: 'options',
options: [
{
name: 'API Token',
value: 'apiToken',
},
{
name: 'OAuth2',
value: 'oAuth2',
},
],
default: 'apiToken',
description: 'Method of authentication.',
},
{
displayName: 'Incoming Authentication',
name: 'incomingAuthentication',
type: 'options',
options: [
{
name: 'Basic Auth',
@@ -91,7 +126,7 @@ export class PipedriveTrigger implements INodeType {
},
],
default: 'none',
description: 'If authentication should be activated for the webhook (makes it more scure).',
description: 'If authentication should be activated for the webhook (makes it more secure).',
},
{
displayName: 'Action',
@@ -218,7 +253,7 @@ export class PipedriveTrigger implements INodeType {
},
async create(this: IHookFunctions): Promise<boolean> {
const webhookUrl = this.getNodeWebhookUrl('default');
const authentication = this.getNodeParameter('authentication', 0) as string;
const incomingAuthentication = this.getNodeParameter('incomingAuthentication', 0) as string;
const eventAction = this.getNodeParameter('action') as string;
const eventObject = this.getNodeParameter('object') as string;
@@ -232,7 +267,7 @@ export class PipedriveTrigger implements INodeType {
http_auth_password: undefined as string | undefined,
};
if (authentication === 'basicAuth') {
if (incomingAuthentication === 'basicAuth') {
const httpBasicAuth = this.getCredentials('httpBasicAuth');
if (httpBasicAuth === undefined || !httpBasicAuth.user || !httpBasicAuth.password) {
@@ -285,9 +320,9 @@ export class PipedriveTrigger implements INodeType {
const resp = this.getResponseObject();
const realm = 'Webhook';
const authentication = this.getNodeParameter('authentication', 0) as string;
const incomingAuthentication = this.getNodeParameter('incomingAuthentication', 0) as string;
if (authentication === 'basicAuth') {
if (incomingAuthentication === 'basicAuth') {
// Basic authorization is needed to call webhook
const httpBasicAuth = this.getCredentials('httpBasicAuth');