Merge pull request #1359 from Bilb/enable-sender-keys-groups

_locales/de/messages.json

@@ -1441,9 +1441,6 @@
     "displayNameEmpty": {
         "message": "Bitte wählen Sie einen Anzeigenamen"
     },
-    "newClosedGroupDescription": {
-        "message": "Geschlossene Gruppen unterstützen bis zu 10 Mitglieder und bieten den gleichen Schutz der Privatsphäre wie Einzelgespräche."
-    },
     "openGroupURL": {
         "message": "Gruppen-URL öffnen"
     },
@@ -1463,6 +1460,6 @@
         "message": "Bitte wählen Sie mindestens zwei Gruppenmitglieder aus."
     },
     "closedGroupMaxSize": {
-        "message": "Eine geschlossene Gruppe kann maximal zehn Mitglieder haben."
+        "message": "Eine geschlossene Gruppe kann maximal zwanzig Mitglieder haben."
     }
 }

_locales/en/messages.json

@@ -2124,10 +2124,6 @@
     "message": "New Closed Group",
     "androidKey": "activity_create_closed_group_title"
   },
-  "newClosedGroupDescription": {
-    "message": "Closed groups support up to 10 members and provide the same privacy protections as one-on-one sessions.",
-    "androidKey": "activity_create_closed_group_explanation"
-  },
   "createClosedGroupNamePrompt": {
     "message": "Group Name",
     "androidKey": "GroupCreateActivity_group_name_hint",
@@ -2178,7 +2174,7 @@
     "androidKey": "activity_create_closed_group_not_enough_group_members_error"
   },
   "closedGroupMaxSize": {
-    "message": "A closed group cannot have more than 10 members",
+    "message": "A closed group cannot have more than 20 members",
     "androidKey": "activity_create_closed_group_too_many_group_members_error"
   },
   "noBlockedContacts": {

_locales/es/messages.json

@@ -1378,9 +1378,6 @@
     "newClosedGroup": {
         "message": "Nuevo grupo cerrado"
     },
-    "newClosedGroupDescription": {
-        "message": "Los grupos cerrados admiten hasta 10 miembros y brindan las mismas protecciones de privacidad que las sesiones individuales."
-    },
     "createClosedGroupNamePrompt": {
         "message": "Nombre Del Grupo"
     },
@@ -1409,6 +1406,6 @@
         "message": "Por favor, elige al menos 2 miembros del grupo"
     },
     "closedGroupMaxSize": {
-        "message": "Un grupo cerrado no puede tener más de 10 miembros"
+        "message": "Un grupo cerrado no puede tener más de 20 miembros"
     }
 }

_locales/fr/messages.json

@@ -1378,9 +1378,6 @@
     "newClosedGroup": {
         "message": "Nouveau groupe privé"
     },
-    "newClosedGroupDescription": {
-        "message": "Les groupes privés prennent en charge jusqu'à 10 membres et offrent le même niveau de confidentialité que les sessions individuelles."
-    },
     "createClosedGroupNamePrompt": {
         "message": "Nom du groupe"
     },
@@ -1409,7 +1406,7 @@
         "message": "Veuillez sélectionner au moins 2 membres"
     },
     "closedGroupMaxSize": {
-        "message": "Un groupe privé ne peut pas avoir plus de 10 membres"
+        "message": "Un groupe privé ne peut pas avoir plus de 20 membres"
     },
     "contextMenuNoSuggestions": {
         "message": "Pas de suggestions",

_locales/id/messages.json

@@ -1372,9 +1372,6 @@
     "newClosedGroup": {
         "message": "Grup tertutup baru"
     },
-    "newClosedGroupDescription": {
-        "message": "Grup tertutup maksimal berisi 10 anggota dan memiliki perlindungan privasi yang sama dengan Session antara dua pengguna"
-    },
     "createClosedGroupNamePrompt": {
         "message": "Nama grup"
     },
@@ -1403,7 +1400,7 @@
         "message": "Pilih setidaknya 2 anggota grup"
     },
     "closedGroupMaxSize": {
-        "message": "Grup tertutup maksimal berisi 10 anggota"
+        "message": "Grup tertutup maksimal berisi 20 anggota"
     },
     "noBlockedContacts": {
         "message": "Tidak ada kontak yang diblokir"

_locales/it/messages.json

@@ -1378,9 +1378,6 @@
     "newClosedGroup": {
         "message": "Nuovo gruppo chiuso"
     },
-    "newClosedGroupDescription": {
-        "message": "I gruppi chiusi supportano fino a 10 membri e forniscono le stesse protezioni per la privacy delle sessioni one-to-one."
-    },
     "createClosedGroupNamePrompt": {
         "message": "Nome Del Gruppo"
     },
@@ -1409,6 +1406,6 @@
         "message": "Scegli almeno 2 membri del gruppo"
     },
     "closedGroupMaxSize": {
-        "message": "Un gruppo chiuso non può avere più di 10 membri"
+        "message": "Un gruppo chiuso non può avere più di 20 membri"
     }
 }

_locales/ja/messages.json

@@ -1378,9 +1378,6 @@
     "newClosedGroup": {
         "message": "新しいクローズドグループ"
     },
-    "newClosedGroupDescription": {
-        "message": "クローズドグループは最大 10 人のメンバーをサポートし、1 対 1 の Session と同じプライバシー保護を提供します。"
-    },
     "createClosedGroupNamePrompt": {
         "message": "グループ名"
     },
@@ -1409,7 +1406,7 @@
         "message": "グループメンバーを少なくとも 2 人選択してください"
     },
     "closedGroupMaxSize": {
-        "message": "閉じたグループは 10 人を超えるメンバーを抱えることはできません"
+        "message": "閉じたグループは 20 人を超えるメンバーを抱えることはできません"
     },
     "noBlockedContacts": {
         "message": "ブロックしている連絡先はありません"

_locales/pl/messages.json

@@ -1378,9 +1378,6 @@
     "newClosedGroup": {
         "message": "Nowa grupa zamknięta"
     },
-    "newClosedGroupDescription": {
-        "message": "Grupy zamknięte obsługują do 10 członków i zapewniają taką samą ochronę prywatności jak sesje jeden na jednego."
-    },
     "createClosedGroupNamePrompt": {
         "message": "Nazwa grupy"
     },
@@ -1406,7 +1403,7 @@
         "message": "Wprowadź krótszą nazwę grupy"
     },
     "closedGroupMaxSize": {
-        "message": "Grupa zamknięta nie może mieć więcej niż 10 członków"
+        "message": "Grupa zamknięta nie może mieć więcej niż 20 członków"
     },
     "noBlockedContacts": {
         "message": "Brak zablokowanych kontaktów"

_locales/pt_BR/messages.json

@@ -1378,9 +1378,6 @@
     "newClosedGroup": {
         "message": "Novo grupo fechado"
     },
-    "newClosedGroupDescription": {
-        "message": "Grupos fechados suportam até 10 membros e fornecem as mesmas proteções de privacidade que as sessões individuais."
-    },
     "createClosedGroupNamePrompt": {
         "message": "Nome Do Grupo"
     },
@@ -1409,6 +1406,6 @@
         "message": "Escolha pelo menos 2 membros do grupo"
     },
     "closedGroupMaxSize": {
-        "message": "Um grupo fechado não pode ter mais de 10 membros"
+        "message": "Um grupo fechado não pode ter mais de 20 membros"
     }
 }

_locales/ru/messages.json

@@ -1378,9 +1378,6 @@
     "newClosedGroup": {
         "message": "Новая закрытая группа"
     },
-    "newClosedGroupDescription": {
-        "message": "Закрытые группы поддерживают до 10 участников и обеспечивают те же меры защиты конфиденциальности, что и сессии один-на-один."
-    },
     "createClosedGroupNamePrompt": {
         "message": "Название Группы"
     },
@@ -1409,7 +1406,7 @@
         "message": "Пожалуйста, выберите как минимум 1 участников группы"
     },
     "closedGroupMaxSize": {
-        "message": "В закрытой группе не может быть больше 10 участников"
+        "message": "В закрытой группе не может быть больше 20 участников"
     },
     "contextMenuNoSuggestions": {
         "message": "No Suggestions",

_locales/vi/messages.json

@@ -1339,9 +1339,6 @@
     "newClosedGroup": {
         "message": "Nhóm kín mới"
     },
-    "newClosedGroupDescription": {
-        "message": "Nhóm kín hỗ trợ tối đa 10 thành viên và cung cấp dịch vụ bảo mật giống như các session một-một"
-    },
     "createClosedGroupNamePrompt": {
         "message": "Tên nhóm"
     },
@@ -1367,6 +1364,6 @@
         "message": "Vui lòng chọn ít nhất 2 thành viên trong nhóm"
     },
     "closedGroupMaxSize": {
-        "message": "Một nhóm kín không thể có nhiều hơn 10 thành viên"
+        "message": "Một nhóm kín không thể có nhiều hơn 20 thành viên"
     }
 }

js/views/invite_contacts_dialog_view.js

@@ -80,12 +80,9 @@
           // Do not trigger an update if there is too many members
           if (
             newMembers.length + existingMembers.length >
-            window.CONSTANTS.SMALL_GROUP_SIZE_LIMIT
+            window.CONSTANTS.MEDIUM_GROUP_SIZE_LIMIT
           ) {
-            const msg = window.i18n(
+            const msg = window.i18n('closedGroupMaxSize');
-              'closedGroupMaxSize',
-              window.CONSTANTS.SMALL_GROUP_SIZE_LIMIT
-            );
 
             window.pushToast({
               title: msg,

preload.js

@@ -86,7 +86,7 @@ window.CONSTANTS = new (function() {
   this.MAX_CONNECTION_DURATION = 5000;
   this.MAX_MESSAGE_BODY_LENGTH = 64 * 1024;
   // Limited due to the proof-of-work requirement
-  this.SMALL_GROUP_SIZE_LIMIT = 10;
+  this.MEDIUM_GROUP_SIZE_LIMIT = 20;
   // Number of seconds to turn on notifications after reconnect/start of app
   this.NOTIFICATION_ENABLE_TIMEOUT_SECONDS = 10;
   this.SESSION_ID_LENGTH = 66;
@@ -490,7 +490,7 @@ if (config.environment.includes('test-integration')) {
     useOnionRequests: false,
     useFileOnionRequests: false,
     debugMessageLogs: true,
-    enableSenderKeys: false,
+    enableSenderKeys: true,
     useMultiDevice: false,
   };
   /* eslint-disable global-require, import/no-extraneous-dependencies */

stylesheets/_session.scss

@@ -656,6 +656,10 @@ label {
     }
   }
 
+  .group-member-list__selection {
+    overflow-y: auto;
+  }
+
   &__centered {
     display: flex;
     flex-direction: column;
@@ -1423,15 +1427,8 @@ input {
     text-align: center;
     padding: 20px;
   }
-
-  // Height at which scroll bar appears on the group member list
-  @media (max-height: 804px) {
-    &__container {
-      overflow-y: visible;
-      max-height: none;
-    }
-  }
 }
+
 .create-group-name-input {
   display: flex;
   justify-content: center;

ts/components/MainViewController.tsx

@@ -83,7 +83,7 @@ async function createClosedGroup(
     });
 
     return;
-  } else if (groupMembers.length >= window.CONSTANTS.SMALL_GROUP_SIZE_LIMIT) {
+  } else if (groupMembers.length >= window.CONSTANTS.MEDIUM_GROUP_SIZE_LIMIT) {
     ToastUtils.push({
       title: window.i18n('closedGroupMaxSize'),
       type: 'error',

ts/components/conversation/CreateGroupDialog.tsx

@@ -8,7 +8,7 @@ import { createLegacyGroup } from '../../session/medium_group';
 declare global {
   interface Window {
     Lodash: any;
-    SMALL_GROUP_SIZE_LIMIT: number;
+    MEDIUM_GROUP_SIZE_LIMIT: number;
   }
 }
 
@@ -200,11 +200,9 @@ export class CreateGroupDialog extends React.Component<Props, State> {
 
     if (
       updatedContacts.filter(d => d.checkmarked).length >
-      window.CONSTANTS.SMALL_GROUP_SIZE_LIMIT - 1
+      window.CONSTANTS.MEDIUM_GROUP_SIZE_LIMIT - 1
     ) {
-      const msg = `${this.props.i18n('closedGroupMaxSize')} ${
+      const msg = this.props.i18n('closedGroupMaxSize');
-        window.CONSTANTS.SMALL_GROUP_SIZE_LIMIT
-      }`;
       this.onShowError(msg);
 
       return;

ts/components/session/SessionClosableOverlay.tsx

@@ -151,7 +151,6 @@ export class SessionClosableOverlay extends React.Component<Props, State> {
       case 'closed-group':
         title = window.i18n('newClosedGroup');
         buttonText = window.i18n('done');
-        descriptionLong = window.i18n('newClosedGroupDescription');
         subtitle = window.i18n('createClosedGroupNamePrompt');
         placeholder = window.i18n('createClosedGroupPlaceholder');
         break;

ts/session/messages/outgoing/content/data/ExpirationTimerUpdateMessage.ts

@@ -37,9 +37,13 @@ export class ExpirationTimerUpdateMessage extends DataMessage {
 
     if (this.groupId) {
       const groupMessage = new SignalService.GroupContext();
-      groupMessage.id = new Uint8Array(
+      let groupIdWithPrefix: string = this.groupId.key;
-        StringUtils.encode(this.groupId.key, 'utf8')
+      if (!this.groupId.key.startsWith(PubKey.PREFIX_GROUP_TEXTSECURE)) {
-      );
+        groupIdWithPrefix = PubKey.PREFIX_GROUP_TEXTSECURE + this.groupId.key;
+      }
+      const encoded = StringUtils.encode(groupIdWithPrefix, 'utf8');
+      const id = new Uint8Array(encoded);
+      groupMessage.id = id;
       groupMessage.type = SignalService.GroupContext.Type.DELIVER;
 
       data.group = groupMessage;