Skip to content

Commit 4bc1987

Browse files
committed
change logger to error
1 parent 2c2d4ce commit 4bc1987

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

spec/CloudCode.spec.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3187,9 +3187,9 @@ describe('sendMail', () => {
31873187

31883188
it('cannot send email without adapter', async () => {
31893189
const logger = require('../lib/logger').logger;
3190-
spyOn(logger, 'warn').and.callFake(() => {});
3190+
spyOn(logger, 'error').and.callFake(() => {});
31913191
await Parse.Cloud.sendMail({});
3192-
expect(logger.warn).toHaveBeenCalledWith(
3192+
expect(logger.error).toHaveBeenCalledWith(
31933193
'Failed to send email because no mail adapter is configured for Parse Server.'
31943194
);
31953195
});

src/cloud-code/Parse.Cloud.js

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -551,12 +551,13 @@ ParseCloud.beforeConnect = function (handler, validationHandler) {
551551
ParseCloud.sendMail = function (data) {
552552
const config = Config.get(Parse.applicationId) || {};
553553
const emailAdapter = config.userController.adapter;
554-
if (emailAdapter) {
555-
return emailAdapter.sendMail(data);
554+
if (!emailAdapter) {
555+
config.loggerController.error(
556+
'Failed to send email because no mail adapter is configured for Parse Server.'
557+
);
558+
return;
556559
}
557-
config.loggerController.warn(
558-
'Failed to send email because no mail adapter is configured for Parse Server.'
559-
);
560+
return emailAdapter.sendMail(data);
560561
};
561562

562563
/**

0 commit comments

Comments
 (0)