chore: Update version

Closes #431, closes #803

README.md

@@ -1,7 +1,7 @@
 # Planka
 #### Elegant open source project tracking.
 
-![David (path)](https://img.shields.io/github/package-json/v/plankanban/planka) ![Docker Pulls](https://img.shields.io/docker/pulls/meltyshev/planka) ![GitHub](https://img.shields.io/github/license/plankanban/planka)
+![David (path)](https://img.shields.io/github/package-json/v/plankanban/planka) ![Docker Pulls](https://img.shields.io/badge/docker_pulls-4M%2B-%23066da5) ![GitHub](https://img.shields.io/github/license/plankanban/planka)
 
 ![](https://raw.githubusercontent.com/plankanban/planka/master/demo.gif)
 

charts/planka/Chart.yaml

@@ -15,13 +15,13 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.2.2
+version: 0.2.4
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to
 # follow Semantic Versioning. They should reflect the version the application is using.
 # It is recommended to use it with quotes.
-appVersion: "1.19.1"
+appVersion: "1.20.0"
 
 dependencies:
   - alias: postgresql

client/.env

@@ -1 +1 @@
-REACT_APP_VERSION=1.19.1
+REACT_APP_VERSION=1.20.0

client/src/actions/cards.js

@@ -57,10 +57,15 @@ updateCard.failure = (id, error) => ({
   },
 });
 
-const handleCardUpdate = (card) => ({
+const handleCardUpdate = (card, isFetched, cardMemberships, cardLabels, tasks, attachments) => ({
   type: ActionTypes.CARD_UPDATE_HANDLE,
   payload: {
     card,
+    isFetched,
+    cardMemberships,
+    cardLabels,
+    tasks,
+    attachments,
   },
 });
 

client/src/constants/ActionTypes.js

@@ -195,9 +195,6 @@ export default {
   CARD_UPDATE__SUCCESS: 'CARD_UPDATE__SUCCESS',
   CARD_UPDATE__FAILURE: 'CARD_UPDATE__FAILURE',
   CARD_UPDATE_HANDLE: 'CARD_UPDATE_HANDLE',
-  CARD_TRANSFER: 'CARD_TRANSFER',
-  CARD_TRANSFER__SUCCESS: 'CARD_TRANSFER__SUCCESS',
-  CARD_TRANSFER__FAILURE: 'CARD_TRANSFER__FAILURE',
   CARD_DUPLICATE: 'CARD_DUPLICATE',
   CARD_DUPLICATE__SUCCESS: 'CARD_DUPLICATE__SUCCESS',
   CARD_DUPLICATE__FAILURE: 'CARD_DUPLICATE__FAILURE',

client/src/locales/de/core.js

@@ -192,6 +192,8 @@ export default {
       deleteTask: 'Aufgabe löschen',
       deleteTask_title: 'Aufgabe löschen',
       deleteUser: 'Benutzer löschen',
+      duplicate: 'Kopieren',
+      duplicateCard_title: 'Karte kopieren',
       edit: 'Bearbeiten',
       editDueDate_title: 'Fälligkeitsdatum bearbeiten',
       editDescription_title: 'Beschreibung ändern',

client/src/locales/hu/core.js

@@ -15,6 +15,16 @@ export default {
 
   translation: {
     common: {
+      aboutPlanka: 'Plankáról',
+      account: 'Fiók',
+      actions: 'Műveletek',
+      addAttachment_title: 'Melléklet hozzáadása',
+      addComment: 'Megjegyzés hozzáadása',
+      addManager_title: 'Menedzser hozzáadása',
+      addMember_title: 'Tag létrehozása',
+      addUser_title: 'Felhasználó hozzáadása',
+      administrator: 'Rendszergazda',
+      all: 'Összes',
       allChangesWillBeAutomaticallySavedAfterConnectionRestored:
         'Az összes változás automatikusan mentésre kerül a kapcsolat helyreállása után.',
       areYouSureYouWantToDeleteThisAttachment: 'Biztosan törölni szeretné ezt a mellékletet?',
@@ -29,7 +39,7 @@ export default {
       areYouSureYouWantToLeaveBoard: 'Biztosan el akarja hagyni a táblát?',
       areYouSureYouWantToLeaveProject: 'Biztosan el akarja hagyni a projektet?',
       areYouSureYouWantToRemoveThisManagerFromProject:
-        'Biztosan eltávolítja ezt a vezetőt a projektből?',
+        'Biztosan eltávolítja ezt a menedzsert a projektből?',
       areYouSureYouWantToRemoveThisMemberFromBoard: 'Biztosan eltávolítja ezt a tagot a tábláról?',
       attachment: 'Melléklet',
       attachments: 'Mellékletek',
@@ -44,6 +54,7 @@ export default {
       cardNotFound_title: 'Kártya nem található',
       cardOrActionAreDeleted: 'A kártya vagy művelet törölve lett.',
       color: 'Szín',
+      copy_inline: 'másolás',
       createBoard_title: 'Tábla létrehozása',
       createLabel_title: 'Címke létrehozása',
       createNewOneOrSelectExistingOne: 'Hozzon létre egy újat, vagy válasszon ki egy meglévőt.',
@@ -52,6 +63,7 @@ export default {
       currentPassword: 'Jelenlegi jelszó',
       dangerZone_title: 'Veszélyzóna',
       date: 'Dátum',
+      dueDate: 'Határidő',
       dueDate_title: 'Esedékesség dátuma',
       deleteAttachment_title: 'Melléklet törlése',
       deleteBoard_title: 'Tábla törlése',
@@ -97,13 +109,17 @@ export default {
       language: 'Nyelv',
       leaveBoard_title: 'Tábla elhagyása',
       leaveProject_title: 'Projekt elhagyása',
+      linkIsCopied: 'Link másolva',
       list: 'Lista',
       listActions_title: 'Lista műveletek',
-      managers: 'Vezetők',
+      managers: 'Menedzserek',
+      managerActions_title: 'Menedzseri műveletek',
       members: 'Tagok',
+      memberActions_title: 'Tag műveletek',
       minutes: 'Percek',
       moveCard_title: 'Kártya áthelyezése',
       name: 'Név',
+      newestFirst: 'Újabbak előre',
       newEmail: 'Új e-mail',
       newPassword: 'Új jelszó',
       newUsername: 'Új felhasználónév',
@@ -113,6 +129,7 @@ export default {
       noProjects: 'Nincsenek projektek',
       notifications: 'Értesítések',
       noUnreadNotifications: 'Nincsenek olvasatlan értesítések.',
+      oldestFirst: 'Régebbi előre',
       openBoard_title: 'Tábla megnyitása',
       optional_inline: 'opcionális',
       organization: 'Szervezet',
@@ -122,17 +139,19 @@ export default {
         'Tipp: nyomja meg a Ctrl-V (Cmd-V a Mac-en) billentyűkombinációt a vágólapról történő melléklet hozzáadásához.',
       project: 'Projekt',
       projectNotFound_title: 'Projekt nem található',
-      removeManager_title: 'Vezető eltávolítása',
+      removeManager_title: 'Menedzser eltávolítása',
       removeMember_title: 'Tag eltávolítása',
       searchLabels: 'Címkék keresése...',
       searchMembers: 'Tagok keresése...',
       searchUsers: 'Felhasználók keresése...',
+      searchCards: 'Kártyák keresése..',
       seconds: 'Másodpercek',
       selectBoard: 'Válassza ki a táblát',
       selectList: 'Válassza ki a listát',
       selectPermissions_title: 'Jogosultságok kiválasztása',
       selectProject: 'Válassza ki a projektet',
       settings: 'Beállítások',
+      sortList_title: 'Rendezés listában',
       stopwatch: 'Stopper',
       subscribeToMyOwnCardsByDefault: 'Alapértelmezés szerint feliratkozás a saját kártyáimra',
       taskActions_title: 'Feladat műveletek',
@@ -170,6 +189,7 @@ export default {
       addTask: 'Feladat hozzáadása',
       addToCard: 'Kártyához hozzáadása',
       addUser: 'Felhasználó hozzáadása',
+      copyLink_title: 'Link másolása',
       createBoard: 'Tábla létrehozása',
       createFile: 'Fájl létrehozása',
       createLabel: 'Címke létrehozása',
@@ -191,6 +211,8 @@ export default {
       deleteTask: 'Feladat törlése',
       deleteTask_title: 'Feladat törlése',
       deleteUser: 'Felhasználó törlése',
+      duplicate: 'Másolás',
+      duplicateCard_title: 'Kártya másolása',
       edit: 'Szerkesztés',
       editDueDate_title: 'Esedékesség dátumának szerkesztése',
       editDescription_title: 'Leírás szerkesztése',
@@ -214,12 +236,13 @@ export default {
       removeCover_title: 'Borító eltávolítása',
       removeFromBoard: 'Eltávolítás a tábláról',
       removeFromProject: 'Eltávolítás a projektről',
-      removeManager: 'Vezető eltávolítása',
+      removeManager: 'Menedzser eltávolítása',
       removeMember: 'Tag eltávolítása',
       save: 'Mentés',
       showAllAttachments: 'Összes melléklet megjelenítése ({{hidden}} rejtve)',
       showDetails: 'Részletek megjelenítése',
       showFewerAttachments: 'Kevesebb melléklet megjelenítése',
+      sortList_title: 'Rendezés listában',
       start: 'Indítás',
       stop: 'Megállítás',
       subscribe: 'Feliratkozás',

client/src/models/Attachment.js

@@ -28,6 +28,7 @@ export default class extends BaseModel {
       case ActionTypes.CORE_INITIALIZE:
       case ActionTypes.PROJECT_MANAGER_CREATE_HANDLE:
       case ActionTypes.BOARD_MEMBERSHIP_CREATE_HANDLE:
+      case ActionTypes.CARD_UPDATE_HANDLE:
         if (payload.attachments) {
           payload.attachments.forEach((attachment) => {
             Attachment.upsert(attachment);

client/src/models/Board.js

@@ -1,3 +1,4 @@
+import orderBy from 'lodash/orderBy';
 import { attr, fk, many } from 'redux-orm';
 
 import BaseModel from './BaseModel';
@@ -218,10 +219,6 @@ export default class extends BaseModel {
     }
   }
 
-  getOrderedMembershipsQuerySet() {
-    return this.memberships.orderBy('createdAt');
-  }
-
   getOrderedLabelsQuerySet() {
     return this.labels.orderBy('position');
   }
@@ -230,6 +227,12 @@ export default class extends BaseModel {
     return this.lists.orderBy('position');
   }
 
+  getOrderedMembershipsModelArray() {
+    return orderBy(this.memberships.toModelArray(), (boardMembershipModel) =>
+      boardMembershipModel.user.name.toLocaleLowerCase(),
+    );
+  }
+
   getMembershipModelForUser(userId) {
     return this.memberships
       .filter({

client/src/models/Card.js

@@ -180,7 +180,6 @@ export default class extends BaseModel {
         break;
       case ActionTypes.CARD_CREATE:
       case ActionTypes.CARD_UPDATE__SUCCESS:
-      case ActionTypes.CARD_UPDATE_HANDLE:
         Card.upsert(payload.card);
 
         break;
@@ -202,8 +201,40 @@ export default class extends BaseModel {
 
         break;
       }
-      case ActionTypes.CARD_UPDATE:
-        Card.withId(payload.id).update(payload.data);
+      case ActionTypes.CARD_UPDATE: {
+        const cardModel = Card.withId(payload.id);
+
+        // TODO: introduce separate action?
+        if (payload.data.boardId && payload.data.boardId !== cardModel.boardId) {
+          cardModel.deleteWithRelated();
+        } else {
+          cardModel.update(payload.data);
+        }
+
+        break;
+      }
+      case ActionTypes.CARD_UPDATE_HANDLE:
+        if (payload.isFetched) {
+          const cardModel = Card.withId(payload.card.id);
+
+          if (cardModel) {
+            cardModel.deleteWithRelated();
+          }
+        }
+
+        Card.upsert(payload.card);
+
+        if (payload.cardMemberships) {
+          payload.cardMemberships.forEach(({ cardId, userId }) => {
+            Card.withId(cardId).users.add(userId);
+          });
+        }
+
+        if (payload.cardLabels) {
+          payload.cardLabels.forEach(({ cardId, labelId }) => {
+            Card.withId(cardId).labels.add(labelId);
+          });
+        }
 
         break;
       case ActionTypes.CARD_DUPLICATE: {

client/src/models/Task.js

@@ -27,6 +27,7 @@ export default class extends BaseModel {
       case ActionTypes.CORE_INITIALIZE:
       case ActionTypes.PROJECT_MANAGER_CREATE_HANDLE:
       case ActionTypes.BOARD_MEMBERSHIP_CREATE_HANDLE:
+      case ActionTypes.CARD_UPDATE_HANDLE:
         if (payload.tasks) {
           payload.tasks.forEach((task) => {
             Task.upsert(task);

client/src/models/User.js

@@ -298,7 +298,7 @@ export default class extends BaseModel {
   static getOrderedUndeletedQuerySet() {
     return this.filter({
       deletedAt: null,
-    }).orderBy('createdAt');
+    }).orderBy((user) => user.name.toLocaleLowerCase());
   }
 
   getOrderedProjectManagersQuerySet() {

client/src/sagas/core/services/cards.js

@@ -81,9 +81,30 @@ export function* updateCurrentCard(data) {
   yield call(updateCard, cardId, data);
 }
 
-// TODO: handle card transfer
 export function* handleCardUpdate(card) {
-  yield put(actions.handleCardUpdate(card));
+  let fetch = false;
+  if (card.boardId) {
+    const prevCard = yield select(selectors.selectCardById, card.id);
+    fetch = !prevCard || prevCard.boardId !== card.boardId;
+  }
+
+  let cardMemberships;
+  let cardLabels;
+  let tasks;
+  let attachments;
+
+  if (fetch) {
+    try {
+      ({
+        item: card, // eslint-disable-line no-param-reassign
+        included: { cardMemberships, cardLabels, tasks, attachments },
+      } = yield call(request, api.getCard, card.id));
+    } catch (error) {
+      fetch = false;
+    }
+  }
+
+  yield put(actions.handleCardUpdate(card, fetch, cardMemberships, cardLabels, tasks, attachments));
 }
 
 export function* moveCard(id, listId, index = 0) {

client/src/selectors/boards.js

@@ -55,10 +55,7 @@ export const selectMembershipsForCurrentBoard = createSelector(
       return boardModel;
     }
 
-    return boardModel
-      .getOrderedMembershipsQuerySet()
-      .toModelArray()
-      .map((boardMembershipModel) => ({
+    return boardModel.getOrderedMembershipsModelArray().map((boardMembershipModel) => ({
       ...boardMembershipModel.ref,
       isPersisted: !isLocalId(boardMembershipModel.id),
       user: {

docker-compose-dev.yml

@@ -29,7 +29,11 @@ services:
       # - OIDC_ISSUER=
       # - OIDC_CLIENT_ID=
       # - OIDC_CLIENT_SECRET=
+      # - OIDC_ID_TOKEN_SIGNED_RESPONSE_ALG=
+      # - OIDC_USERINFO_SIGNED_RESPONSE_ALG=
       # - OIDC_SCOPES=openid email profile
+      # - OIDC_RESPONSE_MODE=fragment
+      # - OIDC_USE_DEFAULT_RESPONSE_MODE=true
       # - OIDC_ADMIN_ROLES=admin
       # - OIDC_EMAIL_ATTRIBUTE=email
       # - OIDC_NAME_ATTRIBUTE=name

docker-compose.yml

@@ -36,7 +36,11 @@ services:
       # - OIDC_ISSUER=
       # - OIDC_CLIENT_ID=
       # - OIDC_CLIENT_SECRET=
+      # - OIDC_ID_TOKEN_SIGNED_RESPONSE_ALG=
+      # - OIDC_USERINFO_SIGNED_RESPONSE_ALG=
       # - OIDC_SCOPES=openid email profile
+      # - OIDC_RESPONSE_MODE=fragment
+      # - OIDC_USE_DEFAULT_RESPONSE_MODE=true
       # - OIDC_ADMIN_ROLES=admin
       # - OIDC_EMAIL_ATTRIBUTE=email
       # - OIDC_NAME_ATTRIBUTE=name

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "planka",
-  "version": "1.19.1",
+  "version": "1.20.0",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "planka",
-      "version": "1.19.1",
+      "version": "1.20.0",
       "hasInstallScript": true,
       "license": "AGPL-3.0",
       "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "planka",
-  "version": "1.19.1",
+  "version": "1.20.0",
   "private": true,
   "homepage": "https://plankanban.github.io/planka",
   "repository": {

server/.env.sample

@@ -27,7 +27,11 @@ SECRET_KEY=notsecretkey
 # OIDC_ISSUER=
 # OIDC_CLIENT_ID=
 # OIDC_CLIENT_SECRET=
+# OIDC_ID_TOKEN_SIGNED_RESPONSE_ALG=
+# OIDC_USERINFO_SIGNED_RESPONSE_ALG=
 # OIDC_SCOPES=openid email profile
+# OIDC_RESPONSE_MODE=fragment
+# OIDC_USE_DEFAULT_RESPONSE_MODE=true
 # OIDC_ADMIN_ROLES=admin
 # OIDC_EMAIL_ATTRIBUTE=email
 # OIDC_NAME_ATTRIBUTE=name

server/api/controllers/access-tokens/exchange-using-oidc.js

@@ -4,6 +4,9 @@ const Errors = {
   INVALID_CODE_OR_NONCE: {
     invalidCodeOrNonce: 'Invalid code or nonce',
   },
+  INVALID_USERINFO_SIGNATURE: {
+    invalidUserinfoSignature: 'Invalid signature on userinfo due to client misconfiguration',
+  },
   EMAIL_ALREADY_IN_USE: {
     emailAlreadyInUse: 'Email already in use',
   },
@@ -31,6 +34,9 @@ module.exports = {
     invalidCodeOrNonce: {
       responseType: 'unauthorized',
     },
+    invalidUserinfoSignature: {
+      responseType: 'unauthorized',
+    },
     emailAlreadyInUse: {
       responseType: 'conflict',
     },
@@ -51,6 +57,7 @@ module.exports = {
         sails.log.warn(`Invalid code or nonce! (IP: ${remoteAddress})`);
         return Errors.INVALID_CODE_OR_NONCE;
       })
+      .intercept('invalidUserinfoSignature', () => Errors.INVALID_USERINFO_SIGNATURE)
       .intercept('emailAlreadyInUse', () => Errors.EMAIL_ALREADY_IN_USE)
       .intercept('usernameAlreadyInUse', () => Errors.USERNAME_ALREADY_IN_USE)
       .intercept('missingValues', () => Errors.MISSING_VALUES);

server/api/controllers/show-config.js

@@ -4,11 +4,16 @@ module.exports = {
     if (sails.hooks.oidc.isActive()) {
       const oidcClient = sails.hooks.oidc.getClient();
 
-      oidc = {
-        authorizationUrl: oidcClient.authorizationUrl({
+      const authorizationUrlParams = {
         scope: sails.config.custom.oidcScopes,
-          response_mode: 'fragment',
-        }),
+      };
+
+      if (!sails.config.custom.oidcUseDefaultResponseMode) {
+        authorizationUrlParams.response_mode = sails.config.custom.oidcResponseMode;
+      }
+
+      oidc = {
+        authorizationUrl: oidcClient.authorizationUrl(authorizationUrlParams),
         endSessionUrl: oidcClient.issuer.end_session_endpoint ? oidcClient.endSessionUrl({}) : null,
         isEnforced: sails.config.custom.oidcEnforced,
       };

server/api/helpers/cards/update-one.js

@@ -204,6 +204,15 @@ module.exports = {
           ),
         );
 
+        sails.sockets.broadcast(
+          `board:${inputs.record.boardId}`,
+          'cardDelete', // TODO: introduce separate event
+          {
+            item: inputs.record,
+          },
+          inputs.request,
+        );
+
         sails.sockets.broadcast(`board:${card.boardId}`, 'cardUpdate', {
           item: card,
         });

server/api/helpers/users/get-or-create-one-using-oidc.js

@@ -12,6 +12,7 @@ module.exports = {
 
   exits: {
     invalidCodeOrNonce: {},
+    invalidUserinfoSignature: {},
     missingValues: {},
     emailAlreadyInUse: {},
     usernameAlreadyInUse: {},
@@ -34,6 +35,14 @@ module.exports = {
       );
       userInfo = await client.userinfo(tokenSet);
     } catch (e) {
+      if (
+        e instanceof SyntaxError &&
+        e.message.includes('Unexpected token e in JSON at position 0')
+      ) {
+        sails.log.warn('Error while exchanging OIDC code: userinfo response is signed');
+        throw 'invalidUserinfoSignature';
+      }
+
       sails.log.warn(`Error while exchanging OIDC code: ${e}`);
       throw 'invalidCodeOrNonce';
     }

server/api/helpers/utils/send-webhooks.js

@@ -116,6 +116,7 @@ async function sendWebhook(webhook, event, data, user) {
     user: sails.helpers.utils.jsonifyRecord(user),
   });
 
+  try {
     const response = await fetch(webhook.url, {
       headers,
       body,
@@ -129,6 +130,9 @@ async function sendWebhook(webhook, event, data, user) {
         `Webhook ${webhook.url} failed with status ${response.status} and message: ${message}`,
       );
     }
+  } catch (e) {
+    sails.log.error(`Webhook ${webhook.url} failed with error message: ${e.message}`);
+  }
 }
 
 module.exports = {

server/api/hooks/oidc/index.js

@@ -25,12 +25,19 @@ module.exports = function defineOidcHook(sails) {
 
       const issuer = await openidClient.Issuer.discover(sails.config.custom.oidcIssuer);
 
-      client = new issuer.Client({
+      const metadata = {
         client_id: sails.config.custom.oidcClientId,
         client_secret: sails.config.custom.oidcClientSecret,
         redirect_uris: [sails.config.custom.oidcRedirectUri],
         response_types: ['code'],
-      });
+        userinfo_signed_response_alg: sails.config.custom.oidcUserinfoSignedResponseAlg,
+      };
+
+      if (sails.config.custom.oidcIdTokenSignedResponseAlg) {
+        metadata.id_token_signed_response_alg = sails.config.custom.oidcIdTokenSignedResponseAlg;
+      }
+
+      client = new issuer.Client(metadata);
     },
 
     getClient() {

server/config/custom.js

@@ -39,7 +39,11 @@ module.exports.custom = {
   oidcIssuer: process.env.OIDC_ISSUER,
   oidcClientId: process.env.OIDC_CLIENT_ID,
   oidcClientSecret: process.env.OIDC_CLIENT_SECRET,
+  oidcIdTokenSignedResponseAlg: process.env.OIDC_ID_TOKEN_SIGNED_RESPONSE_ALG,
+  oidcUserinfoSignedResponseAlg: process.env.OIDC_USERINFO_SIGNED_RESPONSE_ALG,
   oidcScopes: process.env.OIDC_SCOPES || 'openid email profile',
+  oidcResponseMode: process.env.OIDC_RESPONSE_MODE || 'fragment',
+  oidcUseDefaultResponseMode: process.env.OIDC_USE_DEFAULT_RESPONSE_MODE === 'true',
   oidcAdminRoles: process.env.OIDC_ADMIN_ROLES ? process.env.OIDC_ADMIN_ROLES.split(',') : [],
   oidcEmailAttribute: process.env.OIDC_EMAIL_ATTRIBUTE || 'email',
   oidcNameAttribute: process.env.OIDC_NAME_ATTRIBUTE || 'name',