Add checking the package name and activity name length

src/database/app.ts

@@ -34,6 +34,8 @@ export type AppModelStatic = typeof Sequelize.Model & {
   new (values?: object, options?: Sequelize.BuildOptions): AppModel;
 }
 
+export const maxPackageNameLength = 255
+
 export const attributes: SequelizeAttributes<AppAttributes> = {
   familyId: {
     ...familyIdColumn,

src/database/appactivity.ts

@@ -32,6 +32,9 @@ export type AppActivityModelStatic = typeof Sequelize.Model & {
   new (values?: object, options?: Sequelize.BuildOptions): AppActivityModel;
 }
 
+export const maxPackageNameLength = 255
+export const maxActivityNameLength = 255
+
 export const attributes: SequelizeAttributes<AppActivityAttributes> = {
   familyId: {
     ...familyIdColumn,

src/function/sync/apply-actions/dispatch-app-logic-action/addinstalledapps.ts

@@ -1,6 +1,6 @@
 /*
  * server component for the TimeLimit App
- * Copyright (C) 2019 Jonas Lochmann
+ * Copyright (C) 2019 - 2020 Jonas Lochmann
  *
  * This program is free software: you can redistribute it and/or modify
  * it under the terms of the GNU Affero General Public License as
@@ -17,14 +17,24 @@
 
 import * as Sequelize from 'sequelize'
 import { AddInstalledAppsAction } from '../../../../action'
-import { AppAttributes } from '../../../../database/app'
+import { AppAttributes, maxPackageNameLength } from '../../../../database/app'
 import { Cache } from '../cache'
+import { ApplyActionException } from '../exception'
 
 export async function dispatchAddInstalledApps ({ deviceId, action, cache }: {
   deviceId: string
   action: AddInstalledAppsAction
   cache: Cache
 }) {
+  action.apps.forEach((app) => {
+    if (app.packageName.length > maxPackageNameLength) {
+      throw new ApplyActionException({
+        staticMessage: 'package name too long',
+        dynamicMessage: 'package name too long: ' + app.packageName
+      })
+    }
+  })
+
   await cache.database.app.destroy({
     where: {
       familyId: cache.familyId,

src/function/sync/apply-actions/dispatch-app-logic-action/updateappactivities.ts

@@ -18,14 +18,31 @@
 import { chunk } from 'lodash'
 import * as Sequelize from 'sequelize'
 import { UpdateAppActivitiesAction } from '../../../../action'
-import { AppActivityAttributes } from '../../../../database/appactivity'
+import { AppActivityAttributes, maxActivityNameLength, maxPackageNameLength } from '../../../../database/appactivity'
 import { Cache } from '../cache'
+import { ApplyActionException } from '../exception'
 
 export async function dispatchUpdateAppActivities ({ deviceId, action, cache }: {
   deviceId: string
   action: UpdateAppActivitiesAction
   cache: Cache
 }) {
+  action.updatedOrAdded.forEach((app) => {
+    if (app.packageName.length > maxPackageNameLength) {
+      throw new ApplyActionException({
+        staticMessage: 'package name too long',
+        dynamicMessage: 'package name too long: ' + app.packageName
+      })
+    }
+
+    if (app.activityName.length > maxActivityNameLength) {
+      throw new ApplyActionException({
+        staticMessage: 'activity name too long',
+        dynamicMessage: 'activity name too long: ' + app.activityName
+      })
+    }
+  })
+
   if (action.updatedOrAdded.length > 0) {
     const chuncks = chunk(action.updatedOrAdded, 500)