message!'),
+ });
+ /* eslint-enable no-underscore-dangle */
+ logs.push('✅ Email sent successfully!', '');
+
+ logs.push(`📬 Message ID: ${info.messageId}`);
+ if (info.response) {
+ logs.push(`📤 Server response: ${info.response.trim()}`);
+ }
+
+ logs.push('', '🎉 Your configuration is working correctly!');
+ } catch (error) {
+ logs.push('❌ Failed to send email!', '');
+
+ if (error.code) {
+ logs.push(`⚠️ Error code: ${error.code}`);
+ }
+ logs.push(`💬 Reason: ${error.message.trim()}`);
+
+ if (error.code === 'EDNS') {
+ logs.push('', '💡 Hint: Check your host setting.');
+ } else if (error.code === 'ETIMEDOUT') {
+ logs.push('', '💡 Hint: Check your host and port settings.');
+ } else if (error.code === 'EAUTH') {
+ logs.push('', '💡 Hint: Check your username and password.');
+ } else if (error.code === 'ESOCKET') {
+ if (error.message.includes('ECONNREFUSED') || error.message.includes('ETIMEDOUT')) {
+ logs.push('', '💡 Hint: Check your host and port settings.');
+ } else if (error.message.includes('wrong version number')) {
+ logs.push('', '💡 Hint: Try toggling "Use secure connection".');
+ } else if (error.message.includes('certificate')) {
+ logs.push('', '💡 Hint: Try toggling "Reject unauthorized TLS certificates".');
+ }
+ }
+ } finally {
+ transporter.close();
+ }
+
+ return {
+ item: sails.helpers.config.presentOne(config),
+ included: {
+ logs,
+ },
+ };
+ },
+};
diff --git a/server/api/controllers/config/update.js b/server/api/controllers/config/update.js
new file mode 100644
index 00000000..48a9dcf0
--- /dev/null
+++ b/server/api/controllers/config/update.js
@@ -0,0 +1,151 @@
+/*!
+ * Copyright (c) 2024 PLANKA Software GmbH
+ * Licensed under the Fair Use License: https://github.com/plankanban/planka/blob/master/LICENSE.md
+ */
+
+/**
+ * @swagger
+ * /config:
+ * patch:
+ * summary: Update application configuration
+ * description: Updates the application configuration. Requires admin privileges.
+ * tags:
+ * - Config
+ * operationId: updateConfig
+ * requestBody:
+ * required: true
+ * content:
+ * application/json:
+ * schema:
+ * type: object
+ * properties:
+ * smtpHost:
+ * type: string
+ * maxLength: 256
+ * nullable: true
+ * description: Hostname or IP address of the SMTP server
+ * example: smtp.example.com
+ * smtpHost:
+ * type: number
+ * minimum: 0
+ * maximum: 65535
+ * nullable: true
+ * description: Port number of the SMTP server
+ * example: 587
+ * smtpName:
+ * type: string
+ * maxLength: 256
+ * nullable: true
+ * description: Client hostname used in the EHLO command for SMTP
+ * example: localhost
+ * smtpSecure:
+ * type: boolean
+ * description: Whether to use a secure connection for SMTP
+ * example: false
+ * smtpTlsRejectUnauthorized:
+ * type: boolean
+ * description: Whether to reject unauthorized or self-signed TLS certificates for SMTP connections
+ * example: true
+ * smtpUser:
+ * type: string
+ * maxLength: 256
+ * nullable: true
+ * description: Username for authenticating with the SMTP server
+ * example: no-reply@example.com
+ * smtpPassword:
+ * type: string
+ * maxLength: 256
+ * nullable: true
+ * description: Password for authenticating with the SMTP server
+ * example: SecurePassword123!
+ * smtpFrom:
+ * type: string
+ * maxLength: 256
+ * nullable: true
+ * description: Default "from" used for outgoing SMTP emails
+ * example: no-reply@example.com
+ * responses:
+ * 200:
+ * description: Configuration updated successfully
+ * content:
+ * application/json:
+ * schema:
+ * type: object
+ * required:
+ * - item
+ * properties:
+ * item:
+ * $ref: '#/components/schemas/Config'
+ */
+
+module.exports = {
+ inputs: {
+ smtpHost: {
+ type: 'string',
+ isNotEmptyString: true,
+ maxLength: 256,
+ allowNull: true,
+ },
+ smtpPort: {
+ type: 'number',
+ min: 0,
+ max: 65535,
+ allowNull: true,
+ },
+ smtpName: {
+ type: 'string',
+ isNotEmptyString: true,
+ maxLength: 256,
+ allowNull: true,
+ },
+ smtpSecure: {
+ type: 'boolean',
+ },
+ smtpTlsRejectUnauthorized: {
+ type: 'boolean',
+ },
+ smtpUser: {
+ type: 'string',
+ isNotEmptyString: true,
+ maxLength: 256,
+ allowNull: true,
+ },
+ smtpPassword: {
+ type: 'string',
+ isNotEmptyString: true,
+ maxLength: 256,
+ allowNull: true,
+ },
+ smtpFrom: {
+ type: 'string',
+ isNotEmptyString: true,
+ maxLength: 256,
+ allowNull: true,
+ },
+ },
+
+ async fn(inputs) {
+ const { currentUser } = this.req;
+
+ const values = _.pick(inputs, [
+ 'smtpHost',
+ 'smtpPort',
+ 'smtpName',
+ 'smtpSecure',
+ 'smtpTlsRejectUnauthorized',
+ 'smtpUser',
+ 'smtpPassword',
+ 'smtpFrom',
+ ]);
+
+ const config = await sails.helpers.config.updateMain.with({
+ values,
+ actorUser: currentUser,
+ request: this.req,
+ });
+
+ return {
+ item: sails.helpers.config.presentOne(config),
+ };
+ },
+};
diff --git a/server/api/controllers/notification-services/test.js b/server/api/controllers/notification-services/test.js
index 2361e73e..ab9e25cb 100644
--- a/server/api/controllers/notification-services/test.js
+++ b/server/api/controllers/notification-services/test.js
@@ -84,10 +84,17 @@ module.exports = {
}
}
- await sails.helpers.notificationServices.testOne.with({
- record: notificationService,
- i18n: this.req.i18n,
+ /* eslint-disable no-underscore-dangle */
+ await sails.helpers.utils.sendNotifications.with({
+ services: [_.pick(notificationService, ['url', 'format'])],
+ title: this.req.i18n.__('Test Title'),
+ bodyByFormat: {
+ text: this.req.i18n.__('This is a test text message!'),
+ markdown: this.req.i18n.__('This is a *test* **markdown** `message`!'),
+ html: this.req.i18n.__('This is a test html message!'),
+ },
});
+ /* eslint-enable no-underscore-dangle */
return {
item: notificationService,
diff --git a/server/api/controllers/users/update-password.js b/server/api/controllers/users/update-password.js
index e84b6715..f138e1a9 100644
--- a/server/api/controllers/users/update-password.js
+++ b/server/api/controllers/users/update-password.js
@@ -54,14 +54,12 @@
* included:
* type: object
* required:
- * - accessTokens
+ * - accessToken
* properties:
- * accessTokens:
- * type: array
+ * accessToken:
+ * type: string
* description: New acces tokens (when updating own password)
- * items:
- * type: string
- * example: eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ4...
+ * example: eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ4...
* 400:
* $ref: '#/components/responses/ValidationError'
* 401:
@@ -180,7 +178,7 @@ module.exports = {
return {
item: sails.helpers.users.presentOne(user, currentUser),
included: {
- accessTokens: [accessToken],
+ accessToken,
},
};
}
diff --git a/server/api/helpers/actions/create-one.js b/server/api/helpers/actions/create-one.js
index 12bda543..bf7bff92 100644
--- a/server/api/helpers/actions/create-one.js
+++ b/server/api/helpers/actions/create-one.js
@@ -154,9 +154,9 @@ module.exports = {
await sails.helpers.notifications.createOne.with({
values: {
action,
+ userId: action.data.user.id,
type: action.type,
data: action.data,
- userId: action.data.user.id,
creatorUser: values.user,
card: values.card,
},
@@ -179,24 +179,20 @@ module.exports = {
const notifiableUserIds = _.union(cardSubscriptionUserIds, boardSubscriptionUserIds);
- await Promise.all(
- notifiableUserIds.map((userId) =>
- sails.helpers.notifications.createOne.with({
- values: {
- userId,
- action,
- type: action.type,
- data: action.data,
- creatorUser: values.user,
- card: values.card,
- },
- project: inputs.project,
- board: inputs.board,
- list: inputs.list,
- webhooks: inputs.webhooks,
- }),
- ),
- );
+ await sails.helpers.notifications.createMany.with({
+ arrayOfValues: notifiableUserIds.map((userId) => ({
+ userId,
+ action,
+ type: action.type,
+ data: action.data,
+ creatorUser: values.user,
+ card: values.card,
+ })),
+ project: inputs.project,
+ board: inputs.board,
+ list: inputs.list,
+ webhooks: inputs.webhooks,
+ });
}
}
diff --git a/server/api/helpers/bootstrap/present-one.js b/server/api/helpers/bootstrap/present-one.js
new file mode 100644
index 00000000..a06269e2
--- /dev/null
+++ b/server/api/helpers/bootstrap/present-one.js
@@ -0,0 +1,29 @@
+/*!
+ * Copyright (c) 2024 PLANKA Software GmbH
+ * Licensed under the Fair Use License: https://github.com/plankanban/planka/blob/master/LICENSE.md
+ */
+
+module.exports = {
+ sync: true,
+
+ inputs: {
+ oidc: {
+ type: 'ref',
+ },
+ user: {
+ type: 'ref',
+ },
+ },
+
+ fn(inputs) {
+ const data = {
+ oidc: inputs.oidc,
+ version: sails.config.custom.version,
+ };
+ if (inputs.user && inputs.user.role === User.Roles.ADMIN) {
+ data.activeUsersLimit = sails.config.custom.activeUsersLimit;
+ }
+
+ return data;
+ },
+};
diff --git a/server/api/helpers/comments/create-one.js b/server/api/helpers/comments/create-one.js
index ffef0191..7f341453 100644
--- a/server/api/helpers/comments/create-one.js
+++ b/server/api/helpers/comments/create-one.js
@@ -124,29 +124,25 @@ module.exports = {
boardSubscriptionUserIds,
);
- await Promise.all(
- notifiableUserIds.map((userId) =>
- sails.helpers.notifications.createOne.with({
- webhooks,
- values: {
- userId,
- comment,
- type: mentionUserIdsSet.has(userId)
- ? Notification.Types.MENTION_IN_COMMENT
- : Notification.Types.COMMENT_CARD,
- data: {
- card: _.pick(values.card, ['name']),
- text: comment.text,
- },
- creatorUser: values.user,
- card: values.card,
- },
- project: inputs.project,
- board: inputs.board,
- list: inputs.list,
- }),
- ),
- );
+ await sails.helpers.notifications.createMany.with({
+ webhooks,
+ arrayOfValues: notifiableUserIds.map((userId) => ({
+ userId,
+ comment,
+ type: mentionUserIdsSet.has(userId)
+ ? Notification.Types.MENTION_IN_COMMENT
+ : Notification.Types.COMMENT_CARD,
+ data: {
+ card: _.pick(values.card, ['name']),
+ text: comment.text,
+ },
+ creatorUser: values.user,
+ card: values.card,
+ })),
+ project: inputs.project,
+ board: inputs.board,
+ list: inputs.list,
+ });
if (values.user.subscribeToCardWhenCommenting) {
let cardSubscription;
diff --git a/server/api/helpers/config/present-one.js b/server/api/helpers/config/present-one.js
index 3789ca17..1a9dc6fd 100644
--- a/server/api/helpers/config/present-one.js
+++ b/server/api/helpers/config/present-one.js
@@ -11,20 +11,17 @@ module.exports = {
type: 'ref',
required: true,
},
- user: {
- type: 'ref',
- },
},
fn(inputs) {
- const data = {
- ...inputs.record,
- version: sails.config.custom.version,
- };
- if (inputs.user && inputs.user.role === User.Roles.ADMIN) {
- data.activeUsersLimit = sails.config.custom.activeUsersLimit;
+ if (sails.config.custom.smtpHost) {
+ return _.omit(inputs.record, Config.SMTP_FIELD_NAMES);
}
- return data;
+ if (inputs.record.smtpPassword) {
+ return _.omit(inputs.record, 'smtpPassword');
+ }
+
+ return inputs.record;
},
};
diff --git a/server/api/helpers/config/update-main.js b/server/api/helpers/config/update-main.js
new file mode 100644
index 00000000..db8d755e
--- /dev/null
+++ b/server/api/helpers/config/update-main.js
@@ -0,0 +1,53 @@
+/*!
+ * Copyright (c) 2024 PLANKA Software GmbH
+ * Licensed under the Fair Use License: https://github.com/plankanban/planka/blob/master/LICENSE.md
+ */
+
+module.exports = {
+ inputs: {
+ values: {
+ type: 'json',
+ required: true,
+ },
+ actorUser: {
+ type: 'ref',
+ required: true,
+ },
+ request: {
+ type: 'ref',
+ },
+ },
+
+ async fn(inputs) {
+ const { values } = inputs;
+
+ const config = await Config.qm.updateOneMain(values);
+
+ const configRelatedUserIds = await sails.helpers.users.getAllIds(User.Roles.ADMIN);
+
+ configRelatedUserIds.forEach((userId) => {
+ sails.sockets.broadcast(
+ `user:${userId}`,
+ 'configUpdate',
+ {
+ item: sails.helpers.config.presentOne(config),
+ },
+ inputs.request,
+ );
+ });
+
+ const webhooks = await Webhook.qm.getAll();
+
+ // TODO: with prevData?
+ sails.helpers.utils.sendWebhooks.with({
+ webhooks,
+ event: Webhook.Events.CONFIG_UPDATE,
+ buildData: () => ({
+ item: sails.helpers.config.presentOne(config),
+ }),
+ user: inputs.actorUser,
+ });
+
+ return config;
+ },
+};
diff --git a/server/api/helpers/notification-services/test-one.js b/server/api/helpers/notification-services/test-one.js
deleted file mode 100644
index 4ee3f9ac..00000000
--- a/server/api/helpers/notification-services/test-one.js
+++ /dev/null
@@ -1,33 +0,0 @@
-/*!
- * Copyright (c) 2024 PLANKA Software GmbH
- * Licensed under the Fair Use License: https://github.com/plankanban/planka/blob/master/LICENSE.md
- */
-
-/* eslint-disable no-underscore-dangle */
-
-module.exports = {
- inputs: {
- record: {
- type: 'ref',
- required: true,
- },
- i18n: {
- type: 'ref',
- required: true,
- },
- },
-
- async fn(inputs) {
- const { i18n } = inputs;
-
- await sails.helpers.utils.sendNotifications.with({
- services: [_.pick(inputs.record, ['url', 'format'])],
- title: i18n.__('Test Title'),
- bodyByFormat: {
- text: i18n.__('This is a test text message!'),
- markdown: i18n.__('This is a *test* **markdown** `message`!'),
- html: i18n.__('This is a test html message'),
- },
- });
- },
-};
diff --git a/server/api/helpers/notifications/create-many.js b/server/api/helpers/notifications/create-many.js
new file mode 100644
index 00000000..3af6dd16
--- /dev/null
+++ b/server/api/helpers/notifications/create-many.js
@@ -0,0 +1,358 @@
+/*!
+ * Copyright (c) 2024 PLANKA Software GmbH
+ * Licensed under the Fair Use License: https://github.com/plankanban/planka/blob/master/LICENSE.md
+ */
+
+const escapeMarkdown = require('escape-markdown');
+const escapeHtml = require('escape-html');
+
+const { mentionMarkupToText } = require('../../../utils/mentions');
+
+const buildTitle = (notification, t) => {
+ switch (notification.type) {
+ case Notification.Types.MOVE_CARD:
+ return t('Card Moved');
+ case Notification.Types.COMMENT_CARD:
+ return t('New Comment');
+ case Notification.Types.ADD_MEMBER_TO_CARD:
+ return t('You Were Added to Card');
+ case Notification.Types.MENTION_IN_COMMENT:
+ return t('You Were Mentioned in Comment');
+ default:
+ return null;
+ }
+};
+
+const buildBodyByFormat = (board, card, notification, actorUser, t) => {
+ const markdownCardLink = `[${escapeMarkdown(card.name)}](${sails.config.custom.baseUrl}/cards/${card.id})`;
+ const htmlCardLink = `${escapeHtml(card.name)}`;
+
+ switch (notification.type) {
+ case Notification.Types.MOVE_CARD: {
+ const fromListName = sails.helpers.lists.makeName(notification.data.fromList);
+ const toListName = sails.helpers.lists.makeName(notification.data.toList);
+
+ return {
+ text: t(
+ '%s moved %s from %s to %s on %s',
+ actorUser.name,
+ card.name,
+ fromListName,
+ toListName,
+ board.name,
+ ),
+ markdown: t(
+ '%s moved %s from %s to %s on %s',
+ escapeMarkdown(actorUser.name),
+ markdownCardLink,
+ `**${escapeMarkdown(fromListName)}**`,
+ `**${escapeMarkdown(toListName)}**`,
+ escapeMarkdown(board.name),
+ ),
+ html: t(
+ '%s moved %s from %s to %s on %s',
+ escapeHtml(actorUser.name),
+ htmlCardLink,
+ `${escapeHtml(fromListName)}`,
+ `${escapeHtml(toListName)}`,
+ escapeHtml(board.name),
+ ),
+ };
+ }
+ case Notification.Types.COMMENT_CARD: {
+ const commentText = _.truncate(mentionMarkupToText(notification.data.text));
+
+ return {
+ text: `${t(
+ '%s left a new comment to %s on %s',
+ actorUser.name,
+ card.name,
+ board.name,
+ )}:\n${commentText}`,
+ markdown: `${t(
+ '%s left a new comment to %s on %s',
+ escapeMarkdown(actorUser.name),
+ markdownCardLink,
+ escapeMarkdown(board.name),
+ )}:\n\n*${escapeMarkdown(commentText)}*`,
+ html: `${t(
+ '%s left a new comment to %s on %s',
+ escapeHtml(actorUser.name),
+ htmlCardLink,
+ escapeHtml(board.name),
+ )}:\n\n${escapeHtml(commentText)}`,
+ };
+ }
+ case Notification.Types.ADD_MEMBER_TO_CARD:
+ return {
+ text: t('%s added you to %s on %s', actorUser.name, card.name, board.name),
+ markdown: t(
+ '%s added you to %s on %s',
+ escapeMarkdown(actorUser.name),
+ markdownCardLink,
+ escapeMarkdown(board.name),
+ ),
+ html: t(
+ '%s added you to %s on %s',
+ escapeHtml(actorUser.name),
+ htmlCardLink,
+ escapeHtml(board.name),
+ ),
+ };
+ case Notification.Types.MENTION_IN_COMMENT: {
+ const commentText = _.truncate(mentionMarkupToText(notification.data.text));
+
+ return {
+ text: `${t(
+ '%s mentioned you in %s on %s',
+ actorUser.name,
+ card.name,
+ board.name,
+ )}:\n${commentText}`,
+ markdown: `${t(
+ '%s mentioned you in %s on %s',
+ escapeMarkdown(actorUser.name),
+ markdownCardLink,
+ escapeMarkdown(board.name),
+ )}:\n\n*${escapeMarkdown(commentText)}*`,
+ html: `${t(
+ '%s mentioned you in %s on %s',
+ escapeHtml(actorUser.name),
+ htmlCardLink,
+ escapeHtml(board.name),
+ )}:\n\n${escapeHtml(commentText)}`,
+ };
+ }
+ default:
+ return null;
+ }
+};
+
+const buildAndSendNotifications = async (services, board, card, notification, actorUser, t) => {
+ await sails.helpers.utils.sendNotifications(
+ services,
+ buildTitle(notification, t),
+ buildBodyByFormat(board, card, notification, actorUser, t),
+ );
+};
+
+// TODO: use templates (views) to build html
+const buildEmail = (board, card, notification, actorUser, notifiableUser, t) => {
+ const cardLink = `${escapeHtml(card.name)}`;
+ const boardLink = `${escapeHtml(board.name)}`;
+
+ let html;
+ switch (notification.type) {
+ case Notification.Types.MOVE_CARD: {
+ const fromListName = sails.helpers.lists.makeName(notification.data.fromList);
+ const toListName = sails.helpers.lists.makeName(notification.data.toList);
+
+ html = `${t( + '%s moved %s from %s to %s on %s', + escapeHtml(actorUser.name), + cardLink, + escapeHtml(fromListName), + escapeHtml(toListName), + boardLink, + )}
`; + + break; + } + case Notification.Types.COMMENT_CARD: + html = `${t( + '%s left a new comment to %s on %s', + escapeHtml(actorUser.name), + cardLink, + boardLink, + )}
${escapeHtml(mentionMarkupToText(notification.data.text))}
`; + + break; + case Notification.Types.ADD_MEMBER_TO_CARD: + html = `${t( + '%s added you to %s on %s', + escapeHtml(actorUser.name), + cardLink, + boardLink, + )}
`; + + break; + case Notification.Types.MENTION_IN_COMMENT: + html = `${t( + '%s mentioned you in %s on %s', + escapeHtml(actorUser.name), + cardLink, + boardLink, + )}
${escapeHtml(mentionMarkupToText(notification.data.text))}
`; + + break; + default: + return null; // TODO: throw error? + } + + return { + html, + to: notifiableUser.email, + subject: buildTitle(notification, t), + }; +}; + +const sendEmails = async (transporter, emails) => { + await Promise.all( + emails.map((email) => + sails.helpers.utils.sendEmail.with({ + ...email, + transporter, + }), + ), + ); + + transporter.close(); +}; + +module.exports = { + inputs: { + arrayOfValues: { + type: 'ref', + required: true, + }, + project: { + type: 'ref', + required: true, + }, + board: { + type: 'ref', + required: true, + }, + list: { + type: 'ref', + required: true, + }, + webhooks: { + type: 'ref', + required: true, + }, + }, + + async fn(inputs) { + const { arrayOfValues } = inputs; + + const ids = await sails.helpers.utils.generateIds(arrayOfValues.length); + const valuesById = {}; + + const notifications = await Notification.qm.create( + arrayOfValues.map((values) => { + const id = ids.shift(); + + const isCommentRelated = + values.type === Notification.Types.COMMENT_CARD || + values.type === Notification.Types.MENTION_IN_COMMENT; + + const nextValues = { + ...values, + id, + creatorUserId: values.creatorUser.id, + boardId: values.card.boardId, + cardId: values.card.id, + }; + + if (isCommentRelated) { + nextValues.commentId = values.comment.id; + } else { + nextValues.actionId = values.action.id; + } + + valuesById[id] = { ...nextValues }; // FIXME: hack + return nextValues; + }), + ); + + notifications.forEach((notification) => { + const values = valuesById[notification.id]; + + sails.sockets.broadcast(`user:${notification.userId}`, 'notificationCreate', { + item: notification, + included: { + users: [sails.helpers.users.presentOne(values.creatorUser, {})], // FIXME: hack + }, + }); + + sails.helpers.utils.sendWebhooks.with({ + webhooks: inputs.webhooks, + event: Webhook.Events.NOTIFICATION_CREATE, + buildData: () => ({ + item: notification, + included: { + projects: [inputs.project], + boards: [inputs.board], + lists: [inputs.list], + cards: [values.card], + ...(notification.commentId + ? { + comments: [values.comment], + } + : { + actions: [values.action], + }), + }, + }), + user: values.creatorUser, + }); + }); + + const notificationsByUserId = _.groupBy(notifications, 'userId'); + const userIds = Object.keys(notificationsByUserId); + + const notificationServices = await NotificationService.qm.getByUserIds(userIds); + const { transporter } = await sails.helpers.utils.makeSmtpTransporter(); + + if (notificationServices.length > 0 || transporter) { + const users = await User.qm.getByIds(userIds); + const userById = _.keyBy(users, 'id'); + + const notificationServicesByUserId = _.groupBy(notificationServices, 'userId'); + + Object.keys(notificationsByUserId).forEach(async (userId) => { + const notifiableUser = userById[userId]; + const t = sails.helpers.utils.makeTranslator(notifiableUser.language); + + const emails = notificationsByUserId[userId].flatMap((notification) => { + const values = valuesById[notification.id]; + + if (notificationServicesByUserId[userId]) { + const services = notificationServicesByUserId[userId].map((notificationService) => + _.pick(notificationService, ['url', 'format']), + ); + + buildAndSendNotifications( + services, + inputs.board, + values.card, + notification, + values.creatorUser, + t, + ); + } + + if (transporter) { + return buildEmail( + inputs.board, + values.card, + notification, + values.creatorUser, + notifiableUser, + t, + ); + } + + return []; + }); + + if (emails.length > 0) { + sendEmails(transporter, emails); + } + }); + } + + return notifications; + }, +}; diff --git a/server/api/helpers/notifications/create-one.js b/server/api/helpers/notifications/create-one.js index c273e074..25505492 100644 --- a/server/api/helpers/notifications/create-one.js +++ b/server/api/helpers/notifications/create-one.js @@ -137,7 +137,15 @@ const buildAndSendNotifications = async (services, board, card, notification, ac }; // TODO: use templates (views) to build html -const buildAndSendEmail = async (board, card, notification, actorUser, notifiableUser, t) => { +const buildAndSendEmail = async ( + transporter, + board, + card, + notification, + actorUser, + notifiableUser, + t, +) => { const cardLink = `${escapeHtml(card.name)}`; const boardLink = `${escapeHtml(board.name)}`; @@ -164,7 +172,7 @@ const buildAndSendEmail = async (board, card, notification, actorUser, notifiabl escapeHtml(actorUser.name), cardLink, boardLink, - )}${escapeHtml(notification.data.text)}
`; + )}${escapeHtml(mentionMarkupToText(notification.data.text))}
`; break; case Notification.Types.ADD_MEMBER_TO_CARD: @@ -182,7 +190,7 @@ const buildAndSendEmail = async (board, card, notification, actorUser, notifiabl escapeHtml(actorUser.name), cardLink, boardLink, - )}${escapeHtml(notification.data.text)}
`; + )}${escapeHtml(mentionMarkupToText(notification.data.text))}
`; break; default: @@ -190,10 +198,13 @@ const buildAndSendEmail = async (board, card, notification, actorUser, notifiabl } await sails.helpers.utils.sendEmail.with({ + transporter, html, to: notifiableUser.email, subject: buildTitle(notification, t), }); + + transporter.close(); }; module.exports = { @@ -223,10 +234,6 @@ module.exports = { async fn(inputs) { const { values } = inputs; - if (values.user) { - values.userId = values.user.id; - } - const isCommentRelated = values.type === Notification.Types.COMMENT_CARD || values.type === Notification.Types.MENTION_IN_COMMENT; @@ -274,9 +281,10 @@ module.exports = { }); const notificationServices = await NotificationService.qm.getByUserId(notification.userId); + const { transporter } = await sails.helpers.utils.makeSmtpTransporter(); - if (notificationServices.length > 0 || sails.hooks.smtp.isEnabled()) { - const notifiableUser = values.user || (await User.qm.getOneById(notification.userId)); + if (notificationServices.length > 0 || transporter) { + const notifiableUser = await User.qm.getOneById(notification.userId); const t = sails.helpers.utils.makeTranslator(notifiableUser.language); if (notificationServices.length > 0) { @@ -294,8 +302,9 @@ module.exports = { ); } - if (sails.hooks.smtp.isEnabled()) { + if (transporter) { buildAndSendEmail( + transporter, inputs.board, values.card, notification, diff --git a/server/api/helpers/utils/make-smtp-transporter.js b/server/api/helpers/utils/make-smtp-transporter.js new file mode 100644 index 00000000..86e89aca --- /dev/null +++ b/server/api/helpers/utils/make-smtp-transporter.js @@ -0,0 +1,61 @@ +/*! + * Copyright (c) 2024 PLANKA Software GmbH + * Licensed under the Fair Use License: https://github.com/plankanban/planka/blob/master/LICENSE.md + */ + +const nodemailer = require('nodemailer'); + +module.exports = { + inputs: { + defaultOptions: { + type: 'json', + }, + }, + + async fn(inputs) { + let config; + let sourceConfig; + + if (sails.config.custom.smtpHost) { + sourceConfig = sails.config.custom; + } else { + config = await Config.qm.getOneMain(); + + if (config.smtpHost) { + sourceConfig = config; + } + } + + if (!sourceConfig) { + return { + config, + transporter: null, + }; + } + + const transporter = nodemailer.createTransport( + { + ...inputs.defaultOptions, + host: sourceConfig.smtpHost, + port: sourceConfig.smtpPort, + name: sourceConfig.smtpName, + secure: sourceConfig.smtpSecure, + auth: sourceConfig.smtpUser && { + user: sourceConfig.smtpUser, + pass: sourceConfig.smtpPassword, + }, + tls: { + rejectUnauthorized: sourceConfig.smtpTlsRejectUnauthorized, + }, + }, + { + from: sourceConfig.smtpFrom, + }, + ); + + return { + transporter, + config, + }; + }, +}; diff --git a/server/api/helpers/utils/send-email.js b/server/api/helpers/utils/send-email.js index a8e53f44..4bac9d70 100644 --- a/server/api/helpers/utils/send-email.js +++ b/server/api/helpers/utils/send-email.js @@ -5,6 +5,10 @@ module.exports = { inputs: { + transporter: { + type: 'ref', + required: true, + }, to: { type: 'string', required: true, @@ -20,13 +24,8 @@ module.exports = { }, async fn(inputs) { - const transporter = sails.hooks.smtp.getTransporter(); // TODO: check if enabled? - try { - const info = await transporter.sendMail({ - ...inputs, - from: sails.config.custom.smtpFrom, - }); + const info = await inputs.transporter.sendMail(inputs); sails.log.info(`Email sent: ${info.messageId}`); } catch (error) { diff --git a/server/api/hooks/oidc/index.js b/server/api/hooks/oidc/index.js index 37c21e65..accc6737 100644 --- a/server/api/hooks/oidc/index.js +++ b/server/api/hooks/oidc/index.js @@ -59,6 +59,28 @@ module.exports = function defineOidcHook(sails) { return client; }, + async getBootstrap() { + const instance = await this.getClient(); + + if (!instance) { + return null; + } + + const authorizationUrlParams = { + scope: sails.config.custom.oidcScopes, + }; + + if (!sails.config.custom.oidcUseDefaultResponseMode) { + authorizationUrlParams.response_mode = sails.config.custom.oidcResponseMode; + } + + return { + authorizationUrl: instance.authorizationUrl(authorizationUrlParams), + endSessionUrl: instance.issuer.end_session_endpoint ? instance.endSessionUrl({}) : null, + isEnforced: sails.config.custom.oidcEnforced, + }; + }, + isEnabled() { return !!sails.config.custom.oidcIssuer; }, diff --git a/server/api/hooks/query-methods/models/Notification.js b/server/api/hooks/query-methods/models/Notification.js index 0eb54f17..1380feea 100644 --- a/server/api/hooks/query-methods/models/Notification.js +++ b/server/api/hooks/query-methods/models/Notification.js @@ -9,6 +9,37 @@ const defaultFind = (criteria) => Notification.find(criteria).sort('id DESC'); /* Query methods */ +const create = (arrayOfValues) => + sails.getDatastore().transaction(async (db) => { + const notifications = await Notification.createEach(arrayOfValues).fetch().usingConnection(db); + const userIds = sails.helpers.utils.mapRecords(notifications, 'userId', true, true); + + if (userIds.length > 0) { + const queryValues = []; + const inValues = userIds.map((userId) => { + queryValues.push(userId); + return `$${queryValues.length}`; + }); + + queryValues.push(LIMIT); + + const query = ` + WITH exceeded_notification AS ( + SELECT id, ROW_NUMBER() OVER (PARTITION BY user_id ORDER BY id DESC) AS rank + FROM notification + WHERE user_id IN (${inValues.join(', ')}) AND is_read = FALSE + ) + UPDATE notification + SET is_read = TRUE + WHERE id IN (SELECT id FROM exceeded_notification WHERE rank > $${queryValues.length}) + `; + + await sails.sendNativeQuery(query, queryValues).usingConnection(db); + } + + return notifications; + }); + const createOne = (values) => { if (values.userId) { return sails.getDatastore().transaction(async (db) => { @@ -26,7 +57,7 @@ const createOne = (values) => { ) UPDATE notification SET is_read = TRUE - WHERE id in (SELECT id FROM exceeded_notification) + WHERE id IN (SELECT id FROM exceeded_notification) `; await sails.sendNativeQuery(query, [values.userId, LIMIT]).usingConnection(db); @@ -66,6 +97,7 @@ const updateOne = (criteria, values) => Notification.updateOne(criteria).set({ . const delete_ = (criteria) => Notification.destroy(criteria).fetch(); module.exports = { + create, createOne, getByIds, getUnreadByUserId, diff --git a/server/api/hooks/query-methods/models/NotificationService.js b/server/api/hooks/query-methods/models/NotificationService.js index cbb99644..0b5c2881 100644 --- a/server/api/hooks/query-methods/models/NotificationService.js +++ b/server/api/hooks/query-methods/models/NotificationService.js @@ -14,6 +14,11 @@ const getByUserId = (userId) => userId, }); +const getByUserIds = (userIds) => + defaultFind({ + userId: userIds, + }); + const getByBoardId = (boardId) => defaultFind({ boardId, @@ -36,6 +41,7 @@ const deleteOne = (criteria) => NotificationService.destroyOne(criteria); module.exports = { createOne, getByUserId, + getByUserIds, getByBoardId, getByBoardIds, getOneById, diff --git a/server/api/hooks/smtp/index.js b/server/api/hooks/smtp/index.js deleted file mode 100644 index 5bb3e8ec..00000000 --- a/server/api/hooks/smtp/index.js +++ /dev/null @@ -1,55 +0,0 @@ -/*! - * Copyright (c) 2024 PLANKA Software GmbH - * Licensed under the Fair Use License: https://github.com/plankanban/planka/blob/master/LICENSE.md - */ - -/** - * smtp hook - * - * @description :: A hook definition. Extends Sails by adding shadow routes, implicit actions, - * and/or initialization logic. - * @docs :: https://sailsjs.com/docs/concepts/extending-sails/hooks - */ - -const nodemailer = require('nodemailer'); - -module.exports = function defineSmtpHook(sails) { - let transporter = null; - - return { - /** - * Runs when this Sails app loads/lifts. - */ - - async initialize() { - if (!this.isEnabled()) { - return; - } - - sails.log.info('Initializing custom hook (`smtp`)'); - - transporter = nodemailer.createTransport({ - pool: true, - host: sails.config.custom.smtpHost, - port: sails.config.custom.smtpPort, - name: sails.config.custom.smtpName, - secure: sails.config.custom.smtpSecure, - auth: sails.config.custom.smtpUser && { - user: sails.config.custom.smtpUser, - pass: sails.config.custom.smtpPassword, - }, - tls: { - rejectUnauthorized: sails.config.custom.smtpTlsRejectUnauthorized, - }, - }); - }, - - getTransporter() { - return transporter; - }, - - isEnabled() { - return !!sails.config.custom.smtpHost; - }, - }; -}; diff --git a/server/api/models/Config.js b/server/api/models/Config.js index 131779ee..1e7f3a20 100644 --- a/server/api/models/Config.js +++ b/server/api/models/Config.js @@ -17,54 +17,131 @@ * Config: * type: object * required: - * - version - * - oidc + * - id + * - isInitialized * properties: - * version: + * id: * type: string - * description: Current version of the PLANKA application - * example: 2.0.0 - * activeUsersLimit: + * description: Unique identifier for the config (always set to '1') + * example: 1 + * smtpHost: + * type: string + * nullable: true + * description: Hostname or IP address of the SMTP server + * example: smtp.example.com + * smtpPort: * type: number * nullable: true - * description: Maximum number of active users allowed (conditionally added for admins if configured) - * example: 100 - * oidc: - * type: object - * required: - * - authorizationUrl - * - endSessionUrl - * - isEnforced + * description: Port number of the SMTP server + * example: 587 + * smtpName: + * type: string * nullable: true - * description: OpenID Connect configuration (null if not configured) - * properties: - * authorizationUrl: - * type: string - * format: uri - * description: OIDC authorization URL for initiating authentication - * example: https://oidc.example.com/auth - * endSessionUrl: - * type: string - * format: uri - * nullable: true - * description: OIDC end session URL for logout (null if not supported by provider) - * example: https://oidc.example.com/logout - * isEnforced: - * type: boolean - * description: Whether OIDC authentication is enforced (users must use OIDC to login) - * example: false + * description: Client hostname used in the EHLO command for SMTP + * example: localhost + * smtpSecure: + * type: boolean + * description: Whether to use a secure connection for SMTP + * example: false + * smtpTlsRejectUnauthorized: + * type: boolean + * description: Whether to reject unauthorized or self-signed TLS certificates for SMTP connections + * example: true + * smtpUser: + * type: string + * nullable: true + * description: Username for authenticating with the SMTP server + * example: no-reply@example.com + * smtpPassword: + * type: string + * nullable: true + * description: Password for authenticating with the SMTP server + * example: SecurePassword123! + * smtpFrom: + * type: string + * nullable: true + * description: Default "from" used for outgoing SMTP emails + * example: no-reply@example.com + * isInitialized: + * type: boolean + * description: Whether the PLANKA instance has been initialized + * example: true + * createdAt: + * type: string + * format: date-time + * nullable: true + * description: When the config was created + * example: 2024-01-01T00:00:00.000Z + * updatedAt: + * type: string + * format: date-time + * nullable: true + * description: When the config was last updated + * example: 2024-01-01T00:00:00.000Z */ const MAIN_ID = '1'; +const SMTP_FIELD_NAMES = [ + 'smtpHost', + 'smtpPort', + 'smtpName', + 'smtpSecure', + 'smtpTlsRejectUnauthorized', + 'smtpUser', + 'smtpPassword', + 'smtpFrom', +]; + module.exports = { MAIN_ID, + SMTP_FIELD_NAMES, attributes: { // ╔═╗╦═╗╦╔╦╗╦╔╦╗╦╦ ╦╔═╗╔═╗ // ╠═╝╠╦╝║║║║║ ║ ║╚╗╔╝║╣ ╚═╗ // ╩ ╩╚═╩╩ ╩╩ ╩ ╩ ╚╝ ╚═╝╚═╝ + smtpHost: { + type: 'string', + allowNull: true, + columnName: 'smtp_host', + }, + smtpPort: { + type: 'number', + allowNull: true, + columnName: 'smtp_port', + }, + smtpName: { + type: 'string', + allowNull: true, + columnName: 'smtp_name', + }, + smtpSecure: { + type: 'boolean', + required: true, + columnName: 'smtp_secure', + }, + smtpTlsRejectUnauthorized: { + type: 'boolean', + required: true, + columnName: 'smtp_tls_reject_unauthorized', + }, + smtpUser: { + type: 'string', + allowNull: true, + columnName: 'smtp_user', + }, + smtpPassword: { + type: 'string', + allowNull: true, + columnName: 'smtp_password', + }, + smtpFrom: { + type: 'string', + allowNull: true, + columnName: 'smtp_from', + }, isInitialized: { type: 'boolean', required: true, diff --git a/server/api/models/Webhook.js b/server/api/models/Webhook.js index 860da9d8..4e73b9a6 100644 --- a/server/api/models/Webhook.js +++ b/server/api/models/Webhook.js @@ -107,6 +107,8 @@ const Events = { COMMENT_UPDATE: 'commentUpdate', COMMENT_DELETE: 'commentDelete', + CONFIG_UPDATE: 'configUpdate', + CUSTOM_FIELD_CREATE: 'customFieldCreate', CUSTOM_FIELD_UPDATE: 'customFieldUpdate', CUSTOM_FIELD_DELETE: 'customFieldDelete', diff --git a/server/config/custom.js b/server/config/custom.js index 29e6533d..5380d97b 100644 --- a/server/config/custom.js +++ b/server/config/custom.js @@ -102,10 +102,10 @@ module.exports.custom = { smtpPort: process.env.SMTP_PORT || 587, smtpName: process.env.SMTP_NAME, smtpSecure: process.env.SMTP_SECURE === 'true', + smtpTlsRejectUnauthorized: process.env.SMTP_TLS_REJECT_UNAUTHORIZED !== 'false', smtpUser: process.env.SMTP_USER, smtpPassword: process.env.SMTP_PASSWORD, smtpFrom: process.env.SMTP_FROM, - smtpTlsRejectUnauthorized: process.env.SMTP_TLS_REJECT_UNAUTHORIZED !== 'false', gravatarBaseUrl: process.env.GRAVATAR_BASE_URL, }; diff --git a/server/config/locales/de-DE.json b/server/config/locales/de-DE.json index ceb546df..d54252ef 100644 --- a/server/config/locales/de-DE.json +++ b/server/config/locales/de-DE.json @@ -5,7 +5,7 @@ "Test Title": "Testtitel", "This is a test text message!": "Dies ist eine Test-Textnachricht!", "This is a *test* **markdown** `message`!": "Dies ist eine *Test*-**Markdown**-`Nachricht`!", - "This is a test htmlmessage": "Dies ist eine Test-HTML-Nachricht",
+ "This is a test html message!": "Dies ist eine Test-HTML-Nachricht!",
"You Were Added to Card": "Sie wurden zur Karte hinzugefügt",
"You Were Mentioned in Comment": "Sie wurden in einem Kommentar erwähnt",
"%s added you to %s on %s": "%s hat Sie zu %s am %s hinzugefügt",
diff --git a/server/config/locales/el-GR.json b/server/config/locales/el-GR.json
index 1293356b..a47964a2 100644
--- a/server/config/locales/el-GR.json
+++ b/server/config/locales/el-GR.json
@@ -5,7 +5,7 @@
"Test Title": "Τίτλος δοκιμής",
"This is a test text message!": "Αυτό είναι ένα δοκιμαστικό μήνυμα!",
"This is a *test* **markdown** `message`!": "Αυτό είναι ένα *δοκιμαστικό* **markdown** `μήνυμα`!",
- "This is a test html message": "Αυτό είναι ένα δοκιμαστικό html μήνυμα",
+ "This is a test html message!": "Αυτό είναι ένα δοκιμαστικό html μήνυμα!",
"You Were Added to Card": "Προστέθηκες στην κάρτα",
"You Were Mentioned in Comment": "Αναφέρθηκες σε σχόλιο",
"%s added you to %s on %s": "%s σε πρόσθεσε στο %s στο %s",
diff --git a/server/config/locales/en-GB.json b/server/config/locales/en-GB.json
index 441064ca..935cf7a9 100644
--- a/server/config/locales/en-GB.json
+++ b/server/config/locales/en-GB.json
@@ -5,7 +5,7 @@
"Test Title": "Test Title",
"This is a test text message!": "This is a test text message!",
"This is a *test* **markdown** `message`!": "This is a *test* **markdown** `message`!",
- "This is a test html message": "This is a test html message",
+ "This is a test html message!": "This is a test html message!",
"You Were Added to Card": "You Were Added to Card",
"You Were Mentioned in Comment": "You Were Mentioned in Comment",
"%s added you to %s on %s": "%s added you to %s on %s",
diff --git a/server/config/locales/en-US.json b/server/config/locales/en-US.json
index 6e1f2026..8c2e8d73 100644
--- a/server/config/locales/en-US.json
+++ b/server/config/locales/en-US.json
@@ -5,7 +5,7 @@
"Test Title": "Test Title",
"This is a test text message!": "This is a test text message!",
"This is a *test* **markdown** `message`!": "This is a *test* **markdown** `message`!",
- "This is a test html message": "This is a test html message",
+ "This is a test html message!": "This is a test html message!",
"You Were Added to Card": "You Were Added to Card",
"You Were Mentioned in Comment": "You Were Mentioned in Comment",
"%s added you to %s on %s": "%s added you to %s on %s",
diff --git a/server/config/locales/es-ES.json b/server/config/locales/es-ES.json
index 5aa353de..0004be75 100644
--- a/server/config/locales/es-ES.json
+++ b/server/config/locales/es-ES.json
@@ -5,7 +5,7 @@
"Test Title": "Título de prueba",
"This is a test text message!": "¡Este es un mensaje de texto de prueba!",
"This is a *test* **markdown** `message`!": "¡Este es un *mensaje* **markdown** `de prueba`!",
- "This is a test html message": "Este es un mensaje html de prueba",
+ "This is a test html message!": "Este es un mensaje html de prueba!",
"You Were Added to Card": "Fuiste añadido a la tarjeta",
"You Were Mentioned in Comment": "Fuiste mencionado en un comentario",
"%s added you to %s on %s": "%s te añadió a %s en %s",
diff --git a/server/config/locales/fi-FI.json b/server/config/locales/fi-FI.json
index 0d74017b..0f3be176 100644
--- a/server/config/locales/fi-FI.json
+++ b/server/config/locales/fi-FI.json
@@ -5,7 +5,7 @@
"Test Title": "Testin otsikko",
"This is a test text message!": "Tämä on testiviesti!",
"This is a *test* **markdown** `message`!": "Tämä on *testi* **markdown** `viesti`!",
- "This is a test html message": "Tämä on testi html viesti",
+ "This is a test html message!": "Tämä on testi html viesti!",
"You Were Added to Card": "Sinut lisättiin korttiin",
"You Were Mentioned in Comment": "Sinut mainittiin kommentissa",
"%s added you to %s on %s": "%s lisäsi sinut kohteeseen %s kohteessa %s",
diff --git a/server/config/locales/fr-FR.json b/server/config/locales/fr-FR.json
index f7accd8e..0c4cb9e2 100644
--- a/server/config/locales/fr-FR.json
+++ b/server/config/locales/fr-FR.json
@@ -5,7 +5,7 @@
"Test Title": "Titre de test",
"This is a test text message!": "Ceci est un message texte de test !",
"This is a *test* **markdown** `message`!": "Ceci est un *message* **markdown** `de test` !",
- "This is a test html message": "Ceci est un test html message",
+ "This is a test html message!": "Ceci est un test html message!",
"You Were Added to Card": "Vous avez été ajouté à la carte",
"You Were Mentioned in Comment": "Vous avez été mentionné dans un commentaire",
"%s added you to %s on %s": "%s vous a ajouté à %s le %s",
diff --git a/server/config/locales/it-IT.json b/server/config/locales/it-IT.json
index ec13f54e..2e82564c 100644
--- a/server/config/locales/it-IT.json
+++ b/server/config/locales/it-IT.json
@@ -5,7 +5,7 @@
"Test Title": "Titolo di test",
"This is a test text message!": "Questo è un messaggio di testo di test!",
"This is a *test* **markdown** `message`!": "Questo è un *test* **markdown** `messaggio`!",
- "This is a test html message": "Questo è un test html messaggio",
+ "This is a test html message!": "Questo è un test html messaggio!",
"You Were Added to Card": "Sei stato aggiunto alla task",
"You Were Mentioned in Comment": "Sei stato menzionato nel commento",
"%s created %s in %s on %s": "%s ha creato %s in %s in %s",
diff --git a/server/config/locales/ru-RU.json b/server/config/locales/ru-RU.json
index fbd67b79..0467f4f0 100644
--- a/server/config/locales/ru-RU.json
+++ b/server/config/locales/ru-RU.json
@@ -5,7 +5,7 @@
"Test Title": "Тестовый заголовок",
"This is a test text message!": "Это тестовое сообщение!",
"This is a *test* **markdown** `message`!": "Это *тестовое* **markdown** `сообщение`!",
- "This is a test html message": "Это тестовое html сообщение",
+ "This is a test html message!": "Это тестовое html сообщение!",
"You Were Added to Card": "Вы были добавлены к карточке",
"You Were Mentioned in Comment": "Вы были упомянуты в комментарии",
"%s added you to %s on %s": "%s добавил(а) вас к %s на %s",
diff --git a/server/config/locales/tr-TR.json b/server/config/locales/tr-TR.json
index 53999cf9..2ee799d3 100644
--- a/server/config/locales/tr-TR.json
+++ b/server/config/locales/tr-TR.json
@@ -5,7 +5,7 @@
"Test Title": "Test başlığı",
"This is a test text message!": "Bu bir test metin mesajıdır!",
"This is a *test* **markdown** `message`!": "Bu bir *test* **markdown** `mesajı`!",
- "This is a test html message": "Bu bir test html mesajı",
+ "This is a test html message!": "Bu bir test html mesajı!",
"You Were Added to Card": "Karta eklendiniz",
"You Were Mentioned in Comment": "Bir yorumda bahsedildiniz",
"%s added you to %s on %s": "%s sizi %s'ye %s tarihinde ekledi",
diff --git a/server/config/locales/uk-UA.json b/server/config/locales/uk-UA.json
index 4d10d7c2..3f30267c 100644
--- a/server/config/locales/uk-UA.json
+++ b/server/config/locales/uk-UA.json
@@ -5,7 +5,7 @@
"Test Title": "Тестовий заголовок",
"This is a test text message!": "Це нове повідомлення!",
"This is a *test* **markdown** `message`!": "Це *тестове* **markdown** `повідомлення`!",
- "This is a test html message": "Це тестове html повідомлення",
+ "This is a test html message!": "Це тестове html повідомлення!",
"You Were Added to Card": "Вас було додано до картки",
"You Were Mentioned in Comment": "Вас було згадано у коментарі",
"%s added you to %s on %s": "%s додав(ла) вас до %s на %s",
diff --git a/server/config/policies.js b/server/config/policies.js
index 61e6bfce..3d290a30 100644
--- a/server/config/policies.js
+++ b/server/config/policies.js
@@ -18,6 +18,10 @@ module.exports.policies = {
'*': ['is-authenticated', 'is-external'],
+ 'config/show': ['is-authenticated', 'is-admin'],
+ 'config/update': ['is-authenticated', 'is-admin'],
+ 'config/test-smtp': ['is-authenticated', 'is-admin'],
+
'webhooks/index': ['is-authenticated', 'is-external', 'is-admin'],
'webhooks/create': ['is-authenticated', 'is-external', 'is-admin'],
'webhooks/update': ['is-authenticated', 'is-external', 'is-admin'],
@@ -35,7 +39,7 @@ module.exports.policies = {
'projects/create': ['is-authenticated', 'is-external', 'is-admin-or-project-owner'],
- 'config/show': true,
+ 'bootstrap/show': true,
'terms/show': true,
'access-tokens/create': true,
'access-tokens/exchange-with-oidc': true,
diff --git a/server/config/routes.js b/server/config/routes.js
index fb683e6b..39fd7974 100644
--- a/server/config/routes.js
+++ b/server/config/routes.js
@@ -62,10 +62,14 @@ function staticDirServer(prefix, dirFn) {
}
module.exports.routes = {
- 'GET /api/config': 'config/show',
+ 'GET /api/bootstrap': 'bootstrap/show',
'GET /api/terms/:type': 'terms/show',
+ 'GET /api/config': 'config/show',
+ 'PATCH /api/config': 'config/update',
+ 'POST /api/config/test-smtp': 'config/test-smtp',
+
'GET /api/webhooks': 'webhooks/index',
'POST /api/webhooks': 'webhooks/create',
'PATCH /api/webhooks/:id': 'webhooks/update',
diff --git a/server/db/migrations/20250917123048_add_ability_to_configure_smtp_via_ui.js b/server/db/migrations/20250917123048_add_ability_to_configure_smtp_via_ui.js
new file mode 100644
index 00000000..625f46ec
--- /dev/null
+++ b/server/db/migrations/20250917123048_add_ability_to_configure_smtp_via_ui.js
@@ -0,0 +1,36 @@
+/*!
+ * Copyright (c) 2024 PLANKA Software GmbH
+ * Licensed under the Fair Use License: https://github.com/plankanban/planka/blob/master/LICENSE.md
+ */
+
+exports.up = async (knex) => {
+ await knex.schema.alterTable('config', (table) => {
+ /* Columns */
+
+ table.text('smtp_host');
+ table.integer('smtp_port');
+ table.text('smtp_name');
+ table.boolean('smtp_secure').notNullable().defaultTo(false);
+ table.boolean('smtp_tls_reject_unauthorized').notNullable().defaultTo(true);
+ table.text('smtp_user');
+ table.text('smtp_password');
+ table.text('smtp_from');
+ });
+
+ return knex.schema.alterTable('config', (table) => {
+ table.boolean('smtp_secure').notNullable().alter();
+ table.boolean('smtp_tls_reject_unauthorized').notNullable().alter();
+ });
+};
+
+exports.down = (knex) =>
+ knex.schema.alterTable('config', (table) => {
+ table.dropColumn('smtp_host');
+ table.dropColumn('smtp_port');
+ table.dropColumn('smtp_name');
+ table.dropColumn('smtp_secure');
+ table.dropColumn('smtp_tls_reject_unauthorized');
+ table.dropColumn('smtp_user');
+ table.dropColumn('smtp_password');
+ table.dropColumn('smtp_from');
+ });