Using async function within synchronous code
I'm using a library called sqs-consumer to process SQS messages. The library is expecting the message handler to look like:
handleMessage: (message, done) =>
// do some work with `message`
done();
On the other hand though I have async code
async function sendEmail()
return await ...
Is there a good way to run the sendEmail
within handleMessage
, wait for it to finish, and then to call done
?
node.js
add a comment |
I'm using a library called sqs-consumer to process SQS messages. The library is expecting the message handler to look like:
handleMessage: (message, done) =>
// do some work with `message`
done();
On the other hand though I have async code
async function sendEmail()
return await ...
Is there a good way to run the sendEmail
within handleMessage
, wait for it to finish, and then to call done
?
node.js
Possible duplicate of async/await implicitly returns promise?
– lucascaro
Nov 15 '18 at 23:35
2
Note thatreturn await
is pretty pointless if that's the onlyawait
you're using in the function - just use a standard function instead, and you can simplyreturn
the plainPromise
– CertainPerformance
Nov 15 '18 at 23:36
add a comment |
I'm using a library called sqs-consumer to process SQS messages. The library is expecting the message handler to look like:
handleMessage: (message, done) =>
// do some work with `message`
done();
On the other hand though I have async code
async function sendEmail()
return await ...
Is there a good way to run the sendEmail
within handleMessage
, wait for it to finish, and then to call done
?
node.js
I'm using a library called sqs-consumer to process SQS messages. The library is expecting the message handler to look like:
handleMessage: (message, done) =>
// do some work with `message`
done();
On the other hand though I have async code
async function sendEmail()
return await ...
Is there a good way to run the sendEmail
within handleMessage
, wait for it to finish, and then to call done
?
node.js
node.js
asked Nov 15 '18 at 23:33
JaeJae
91131123
91131123
Possible duplicate of async/await implicitly returns promise?
– lucascaro
Nov 15 '18 at 23:35
2
Note thatreturn await
is pretty pointless if that's the onlyawait
you're using in the function - just use a standard function instead, and you can simplyreturn
the plainPromise
– CertainPerformance
Nov 15 '18 at 23:36
add a comment |
Possible duplicate of async/await implicitly returns promise?
– lucascaro
Nov 15 '18 at 23:35
2
Note thatreturn await
is pretty pointless if that's the onlyawait
you're using in the function - just use a standard function instead, and you can simplyreturn
the plainPromise
– CertainPerformance
Nov 15 '18 at 23:36
Possible duplicate of async/await implicitly returns promise?
– lucascaro
Nov 15 '18 at 23:35
Possible duplicate of async/await implicitly returns promise?
– lucascaro
Nov 15 '18 at 23:35
2
2
Note that
return await
is pretty pointless if that's the only await
you're using in the function - just use a standard function instead, and you can simply return
the plain Promise
– CertainPerformance
Nov 15 '18 at 23:36
Note that
return await
is pretty pointless if that's the only await
you're using in the function - just use a standard function instead, and you can simply return
the plain Promise
– CertainPerformance
Nov 15 '18 at 23:36
add a comment |
2 Answers
2
active
oldest
votes
You can use your promise-based code inside a callback like this:
handleMessage: (message, done) =>
// do some work with `message`
sendEmail().then(done).catch(...);
In some cases, you could also make a new method for handleMessage()
that actually expects a promise to be returned and works with that promise directly rather than using a plain callback at all.
Also, not that there is no point to using return await ...
at the end of your function. You can just return the promise directly:
So, instead of:
async function sendEmail()
// other code here
return await someFuncThatReturnsPromise();
You can just do:
async function sendEmail()
// other code here
return someFuncThatReturnsPromise();
That extra await
adds no value or utility.
add a comment |
As long as the library doesn't care what you return from handleMessage
you can also use async/await in the handlerMessage function directly:
handleMessage: async (message, done) =>
await sendEmail();
done();
add a comment |
Your Answer
StackExchange.ifUsing("editor", function ()
StackExchange.using("externalEditor", function ()
StackExchange.using("snippets", function ()
StackExchange.snippets.init();
);
);
, "code-snippets");
StackExchange.ready(function()
var channelOptions =
tags: "".split(" "),
id: "1"
;
initTagRenderer("".split(" "), "".split(" "), channelOptions);
StackExchange.using("externalEditor", function()
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled)
StackExchange.using("snippets", function()
createEditor();
);
else
createEditor();
);
function createEditor()
StackExchange.prepareEditor(
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: true,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
imageUploader:
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
,
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
);
);
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53329340%2fusing-async-function-within-synchronous-code%23new-answer', 'question_page');
);
Post as a guest
Required, but never shown
2 Answers
2
active
oldest
votes
2 Answers
2
active
oldest
votes
active
oldest
votes
active
oldest
votes
You can use your promise-based code inside a callback like this:
handleMessage: (message, done) =>
// do some work with `message`
sendEmail().then(done).catch(...);
In some cases, you could also make a new method for handleMessage()
that actually expects a promise to be returned and works with that promise directly rather than using a plain callback at all.
Also, not that there is no point to using return await ...
at the end of your function. You can just return the promise directly:
So, instead of:
async function sendEmail()
// other code here
return await someFuncThatReturnsPromise();
You can just do:
async function sendEmail()
// other code here
return someFuncThatReturnsPromise();
That extra await
adds no value or utility.
add a comment |
You can use your promise-based code inside a callback like this:
handleMessage: (message, done) =>
// do some work with `message`
sendEmail().then(done).catch(...);
In some cases, you could also make a new method for handleMessage()
that actually expects a promise to be returned and works with that promise directly rather than using a plain callback at all.
Also, not that there is no point to using return await ...
at the end of your function. You can just return the promise directly:
So, instead of:
async function sendEmail()
// other code here
return await someFuncThatReturnsPromise();
You can just do:
async function sendEmail()
// other code here
return someFuncThatReturnsPromise();
That extra await
adds no value or utility.
add a comment |
You can use your promise-based code inside a callback like this:
handleMessage: (message, done) =>
// do some work with `message`
sendEmail().then(done).catch(...);
In some cases, you could also make a new method for handleMessage()
that actually expects a promise to be returned and works with that promise directly rather than using a plain callback at all.
Also, not that there is no point to using return await ...
at the end of your function. You can just return the promise directly:
So, instead of:
async function sendEmail()
// other code here
return await someFuncThatReturnsPromise();
You can just do:
async function sendEmail()
// other code here
return someFuncThatReturnsPromise();
That extra await
adds no value or utility.
You can use your promise-based code inside a callback like this:
handleMessage: (message, done) =>
// do some work with `message`
sendEmail().then(done).catch(...);
In some cases, you could also make a new method for handleMessage()
that actually expects a promise to be returned and works with that promise directly rather than using a plain callback at all.
Also, not that there is no point to using return await ...
at the end of your function. You can just return the promise directly:
So, instead of:
async function sendEmail()
// other code here
return await someFuncThatReturnsPromise();
You can just do:
async function sendEmail()
// other code here
return someFuncThatReturnsPromise();
That extra await
adds no value or utility.
answered Nov 15 '18 at 23:35
jfriend00jfriend00
438k55575619
438k55575619
add a comment |
add a comment |
As long as the library doesn't care what you return from handleMessage
you can also use async/await in the handlerMessage function directly:
handleMessage: async (message, done) =>
await sendEmail();
done();
add a comment |
As long as the library doesn't care what you return from handleMessage
you can also use async/await in the handlerMessage function directly:
handleMessage: async (message, done) =>
await sendEmail();
done();
add a comment |
As long as the library doesn't care what you return from handleMessage
you can also use async/await in the handlerMessage function directly:
handleMessage: async (message, done) =>
await sendEmail();
done();
As long as the library doesn't care what you return from handleMessage
you can also use async/await in the handlerMessage function directly:
handleMessage: async (message, done) =>
await sendEmail();
done();
answered Nov 16 '18 at 0:12
PaulproPaulpro
114k15226232
114k15226232
add a comment |
add a comment |
Thanks for contributing an answer to Stack Overflow!
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53329340%2fusing-async-function-within-synchronous-code%23new-answer', 'question_page');
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Possible duplicate of async/await implicitly returns promise?
– lucascaro
Nov 15 '18 at 23:35
2
Note that
return await
is pretty pointless if that's the onlyawait
you're using in the function - just use a standard function instead, and you can simplyreturn
the plainPromise
– CertainPerformance
Nov 15 '18 at 23:36