We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hi there,
I was getting error promise is not a function from https://github.com/disney/terraform-aws-kinesis-firehose-splunk/blob/master/files/kinesis-firehose-cloudwatch-logs-processor.js#L131:
promise is not a function
const response = await client[methodName](args).promise()
While debugging I printed the return of client[methodName](args) and it was of type promise, so I changed it to:
client[methodName](args)
const response = await client[methodName](args)
It solved my problem, so I thought I would let you guys know.
The text was updated successfully, but these errors were encountered:
Digging deeper on this issue, this is happening because it's now using aws-sdk v3:
below a snippet of code using aws-sdk v2:
var AWS = require("aws-sdk") AWS.config.update({region: "eu-west-2"}) client = new AWS.Firehose(); (async function(){ r = await client.listDeliveryStreams({}).promise() console.log(r) })()
Using aws-sdk v3:
const { Firehose } = require("@aws-sdk/client-firehose"); ( async function(){ client = new Firehose({region: "eu-west-2"}) r = await client.listDeliveryStreams({}) console.log(r) })()
Sorry, something went wrong.
Fix #36 - Update Lambda code for AWS SDK v3.x
8ad1224
Fixed in #38
7b12ccb
No branches or pull requests
Hi there,
I was getting error
promise is not a function
from https://github.com/disney/terraform-aws-kinesis-firehose-splunk/blob/master/files/kinesis-firehose-cloudwatch-logs-processor.js#L131:While debugging I printed the return of
client[methodName](args)
and it was of type promise, so I changed it to:It solved my problem, so I thought I would let you guys know.
The text was updated successfully, but these errors were encountered: