16 changed files with 1987 additions and 1422 deletions
--- a/.eslintrc.json
+++ b/.eslintrc.json
@ -88,7 +88,6 @@
  },
  "ignorePatterns": [
    "jest.config.js",
-    "scripts/**/*",
    "webpack.config.js",
    "dist/**/*",
    "public/**/*",
--- a/assets/sass/_vars.scss
+++ b/assets/sass/_vars.scss
@ -30,4 +30,4 @@ $errorColor: desaturate($errorText, 50%);

 // Responsivity
 $mobileThreshold: 632px;
-$desktopThreshold: 940px;
+$desktopThreshold: 632px;
--- a/assets/sass/layout.scss
+++ b/assets/sass/layout.scss
@ -173,15 +173,6 @@ body > header {
 						font-weight: inherit;
 						background: transparent;
 					}
-
-					&:hover {
-						.tip {
-							visibility: visible;
-							opacity: 1;
-							transition: opacity ease-out 100ms;
-							transition-delay: 150ms;
-						}
-					}
 				}

 				button {
@ -407,7 +398,7 @@ form {
 	padding: 8px 16px;
 	text-align: center;

-	.form-field:not(.hidden) {
+	.form-field {
 		display: flex;
 		flex-direction: column;
 		margin: 16px auto;
@ -702,7 +693,6 @@ td.actions {

 	th {
 		border-bottom: 1px solid #39434a;
-		white-space: nowrap;
 	}

 	tr:nth-child(even) {
@ -869,7 +859,37 @@ td.actions {
 	}

 	.content {
-		width: 0;
+		overflow: hidden;
+		white-space: nowrap;
+		padding: 8px;
+	}
+
+	.copy-button {
+		margin: 0;
+		padding: 0;
+		border-radius: 0;
+
+		.feather {
+			--icon-size: 20px;
+			margin: 8px;
+		}
+	}
+}
+
+.copyable-text {
+	display: flex;
+	flex-direction: row;
+	margin: 8px;
+
+	background-color: darken($backgroundColor, 2%);
+	border-radius: 5px;
+	overflow: hidden;
+
+	.title {
+		padding: 8px;
+	}
+
+	.content {
 		flex-grow: 1;
 		overflow: hidden;
 		white-space: nowrap;
@ -919,34 +939,3 @@ td.actions {
 		background: $secondary;
 	}
 }
-
-.table-col-grow {
-	width: 100%;
-}
-
-.pagination {
-	ul {
-		display: flex;
-		flex-direction: row;
-		list-style: none;
-		padding: 8px;
-
-		justify-content: center;
-
-		li {
-			a, &.active, &.ellipsis {
-				display: block;
-				min-width: 40px;
-				height: 40px;
-				padding: 4px;
-
-				line-height: 32px;
-				text-align:center;
-
-				&:hover:not(.active):not(.ellipsis) {
-					background-color: #fff5;
-				}
-			}
-		}
-	}
-}
--- a/config/default.json5
+++ b/config/default.json5
@ -1,75 +1,69 @@
 {
-	app: {
-		name: 'Rainbox Email',
-		contact_email: 'contact@rainbox.email',
-		webmail_url: 'https://rc.rainbox.email',
-	},
-	log_level: "DEV",
-	db_log_level: "ERROR",
-	public_url: "http://localhost:4899",
-	public_websocket_url: "ws://localhost:4899",
-	port: 4899,
-	mysql: {
-		connectionLimit: 10,
-		host: "localhost",
-		user: "root",
-		password: "",
-		database: "rainbox_email",
-		create_database_automatically: false,
-	},
-	redis: {
-		host: "127.0.0.1",
-		port: 6379,
-		prefix: 'rainbox.email',
-	},
-	session: {
-		cookie: {
-			secure: false,
-		},
-	},
-	mail: {
-		host: "127.0.0.1",
-		port: "1025",
-		secure: false,
-		username: "",
-		password: "",
-		allow_invalid_tls: true,
-		from: 'contact@rainbox.email',
-		from_name: 'Rainbox Email',
-		// Unlimited
-		max_public_identities_per_user: -1,
-	},
-	view: {
-		cache: false,
-	},
-	auth: {
-		approval_mode: false,
-		// 30 days
-		name_change_wait_period: 2592000000,
-	},
-	magic_link: {
-		validity_period: 20,
-	},
-	mail_autoconfig: {
-		main_domain: 'localhost',
-		display_name: 'Rainbox Email',
-		display_name_short: 'Rainbox',
-		username: '%EMAILADDRESS%',
-		auth_method: 'password-cleartext',
-		imap: {
-			server: 'localhost',
-			port: '143',
-			method: 'STARTTLS',
-		},
-		pop3: {
-			server: 'localhost',
-			port: '110',
-			method: 'STARTTLS',
-		},
-		smtp: {
-			server: 'localhost',
-			port: '587',
-			method: 'STARTTLS',
-		},
-	},
+    app: {
+        name: 'Rainbox Email',
+        contact_email: 'contact@rainbox.email',
+        webmail_url: 'https://rc.rainbox.email',
+    },
+    log_level: "DEV",
+    db_log_level: "ERROR",
+    public_url: "http://localhost:4899",
+    public_websocket_url: "ws://localhost:4899",
+    port: 4899,
+    mysql: {
+        connectionLimit: 10,
+        host: "localhost",
+        user: "root",
+        password: "",
+        database: "rainbox_email",
+        create_database_automatically: false,
+    },
+    redis: {
+        host: "127.0.0.1",
+        port: 6379,
+        prefix: 'rainbox.email',
+    },
+    session: {
+        cookie: {
+            secure: false,
+        },
+    },
+    mail: {
+        host: "127.0.0.1",
+        port: "1025",
+        secure: false,
+        username: "",
+        password: "",
+        allow_invalid_tls: true,
+        from: 'contact@rainbox.email',
+        from_name: 'Rainbox Email',
+    },
+    view: {
+        cache: false,
+    },
+    approval_mode: false,
+    magic_link: {
+        validity_period: 20,
+    },
+    mail_autoconfig: {
+        main_domain: 'localhost',
+        display_name: 'Rainbox Email',
+        display_name_short: 'Rainbox',
+        username: '%EMAILADDRESS%',
+        auth_method: 'password-cleartext',
+        imap: {
+            server: 'localhost',
+            port: '143',
+            method: 'STARTTLS',
+        },
+        pop3: {
+            server: 'localhost',
+            port: '110',
+            method: 'STARTTLS',
+        },
+        smtp: {
+            server: 'localhost',
+            port: '587',
+            method: 'STARTTLS',
+        },
+    },
 }
--- a/config/production.json5
+++ b/config/production.json5
@ -1,43 +1,41 @@
 {
-	log_level: "DEBUG",
-	db_log_level: "ERROR",
-	public_url: "https://rainbox.email",
-	public_websocket_url: "wss://rainbox.email",
-	session: {
-		cookie: {
-			secure: true,
-		},
-	},
-	mail: {
-		secure: true,
-		allow_invalid_tls: false,
-	},
-	auth: {
-		approval_mode: true,
-	},
-	magic_link: {
-		validity_period: 900,
-	},
-	mail_autoconfig: {
-		main_domain: 'rainbox.email',
-		display_name: 'Rainbox Email',
-		display_name_short: 'Rainbox',
-		username: '%EMAILADDRESS%',
-		auth_method: 'password-cleartext',
-		imap: {
-			server: 'rainbox.email',
-			port: '143',
-			method: 'STARTTLS',
-		},
-		pop3: {
-			server: 'rainbox.email',
-			port: '110',
-			method: 'STARTTLS',
-		},
-		smtp: {
-			server: 'rainbox.email',
-			port: '587',
-			method: 'STARTTLS',
-		},
-	},
+    log_level: "DEBUG",
+    db_log_level: "ERROR",
+    public_url: "https://rainbox.email",
+    public_websocket_url: "wss://rainbox.email",
+    session: {
+        cookie: {
+            secure: true
+        }
+    },
+    mail: {
+        secure: true,
+        allow_invalid_tls: false
+    },
+    approval_mode: true,
+    magic_link: {
+        validity_period: 900,
+    },
+    mail_autoconfig: {
+        main_domain: 'rainbox.email',
+        display_name: 'Rainbox Email',
+        display_name_short: 'Rainbox',
+        username: '%EMAILADDRESS%',
+        auth_method: 'password-cleartext',
+        imap: {
+            server: 'rainbox.email',
+            port: '143',
+            method: 'STARTTLS',
+        },
+        pop3: {
+            server: 'rainbox.email',
+            port: '110',
+            method: 'STARTTLS',
+        },
+        smtp: {
+            server: 'rainbox.email',
+            port: '587',
+            method: 'STARTTLS',
+        },
+    },
 }
--- a/config/test.json5
+++ b/config/test.json5
@ -6,7 +6,4 @@
        database: "rainbox_email_test",
        create_database_automatically: true
    },
-    mail: {
-        max_public_identities_per_user: 1,
-    },
 }
--- a/package.json
+++ b/package.json
@ -1,20 +1,20 @@
 {
  "name": "rainbox.email",
-  "version": "2.4.2",
+  "version": "2.4.0",
  "description": "ISP mail provider manager with mysql and integrated LDAP server",
-  "repository": "https://eternae.ink/ashpie/rainbox.email",
+  "repository": "https://gitlab.com/ArisuOngaku/rainbox.email",
  "author": "Alice Gaudon <alice@gaudon.pro>",
  "main": "dist/main.js",
  "license": "AGPL-3.0-only",
  "scripts": {
    "test": "jest --verbose --runInBand",
-    "clean": "node scripts/clean.js",
-    "prepare-sources": "node scripts/prepare-sources.js",
+    "clean": "(test ! -d dist || rm -r dist)",
+    "prepareSources": "cp package.json src/",
    "compile": "yarn clean && tsc",
-    "build": "yarn prepare-sources && yarn compile && webpack --mode production",
-    "dev": "yarn prepare-sources && concurrently -k -n \"Typescript,Node,Webpack,Maildev\" -p \"[{name}]\" -c \"blue,green,red,yellow\" \"tsc --watch\" \"nodemon\" \"webpack --watch --mode development\" \"maildev\"",
+    "build": "yarn prepareSources && yarn compile && webpack --mode production",
+    "dev": "yarn prepareSources && concurrently -k -n \"Typescript,Node,Webpack,Maildev\" -p \"[{name}]\" -c \"blue,green,red,yellow\" \"tsc --watch\" \"nodemon\" \"webpack --watch --mode development\" \"maildev\"",
    "start": "yarn build && node",
-    "lint": "eslint ."
+    "lint": "eslint . --ext .js,.jsx,.ts,.tsx"
  },
  "devDependencies": {
    "@babel/core": "^7.9.0",
@ -28,7 +28,7 @@
    "@types/jest": "^26.0.4",
    "@types/ldapjs": "^1.0.7",
    "@types/mysql": "^2.15.15",
-    "@types/node": "^15.0.1",
+    "@types/node": "^14.6.3",
    "@types/nodemailer": "^6.4.0",
    "@types/nunjucks": "^3.1.3",
    "@types/ws": "^7.2.6",
@ -40,21 +40,21 @@
    "eslint": "^7.10.0",
    "feather-icons": "^4.28.0",
    "file-loader": "^6.0.0",
-    "imagemin": "^7.0.0",
+    "imagemin": "^7.0.1",
    "imagemin-gifsicle": "^7.0.0",
    "imagemin-mozjpeg": "^9.0.0",
    "imagemin-pngquant": "^9.0.0",
-    "imagemin-svgo": "^9.0.0",
+    "imagemin-svgo": "^8.0.0",
    "img-loader": "^3.0.1",
    "jest": "^26.1.0",
    "maildev": "^1.1.0",
    "mini-css-extract-plugin": "^1.2.1",
+    "node-sass": "^5.0.0",
    "nodemon": "^2.0.3",
-    "sass": "^1.32.12",
    "sass-loader": "^11.0.1",
    "terser-webpack-plugin": "^5.0.3",
    "ts-jest": "^26.1.1",
-    "ts-loader": "^9.1.0",
+    "ts-loader": "^8.0.4",
    "typescript": "^4.0.2",
    "webpack": "^5.3.2",
    "webpack-cli": "^4.1.0"
--- a/scripts/clean.js
+++ b/scripts/clean.js
@ -1,10 +0,0 @@
-const fs = require('fs');
-
-[
-    'dist',
-].forEach(file => {
-    if (fs.existsSync(file)) {
-        console.log('Cleaning', file, '...');
-        fs.rmSync(file, {recursive: true});
-    }
-});
--- a/scripts/prepare-sources.js
+++ b/scripts/prepare-sources.js
@ -1,4 +0,0 @@
-const fs = require('fs');
-const path = require('path');
-
-fs.copyFileSync('package.json', path.join('src', 'package.json'));
--- a/src/App.ts
+++ b/src/App.ts
@ -42,7 +42,6 @@ import AccountController from "swaf/auth/AccountController";
 import packageJson = require('./package.json');
 import AddUsedToMagicLinksMigration from "swaf/auth/magic_link/AddUsedToMagicLinksMigration";
 import MakeMagicLinksSessionNotUniqueMigration from "swaf/auth/magic_link/MakeMagicLinksSessionNotUniqueMigration";
-import AddNameChangedAtToUsersMigration from "swaf/auth/migrations/AddNameChangedAtToUsersMigration";

 export default class App extends Application {
    public constructor(
@ -67,7 +66,6 @@ export default class App extends Application {
            DropLegacyLogsTable,
            AddUsedToMagicLinksMigration,
            MakeMagicLinksSessionNotUniqueMigration,
-            AddNameChangedAtToUsersMigration,
        ];
    }

--- a/src/controllers/AccountMailboxController.ts
+++ b/src/controllers/AccountMailboxController.ts
@ -57,20 +57,21 @@ export default class AccountMailboxController extends Controller {
        });

        // Check whether this identity can be created by this user
-        if (!domain.canCreateAddresses(user)) {
-            throw new ForbiddenHttpError('domain', req.url);
-        }
        if (domain.isPublic()) {
            await Validator.validate({
-                name: new Validator<string>().defined().equals(user.as(UserNameComponent).getName()),
+                name: new Validator<string>().defined().equals(user.as(UserNameComponent).name),
            }, req.body);
            const actualPublicAddressesCount = await mailIdentityComponent.getPublicAddressesCount();
            const maxPublicAddressesCount = mailIdentityComponent.getMaxPublicAddressesCount();
-            if (maxPublicAddressesCount >= 0 && actualPublicAddressesCount >= maxPublicAddressesCount) {
+            if (actualPublicAddressesCount >= maxPublicAddressesCount) {
                req.flash('error', 'You have reached maximum public email addresses.');
                res.redirect(Controller.route('account-mailbox'));
                return;
            }
+        } else {
+            if (!domain.canCreateAddresses(user)) {
+                throw new ForbiddenHttpError('domain', req.url);
+            }
        }

        // Save identity
--- a/src/controllers/backend/AccountBackendController.ts
+++ b/src/controllers/backend/AccountBackendController.ts
@ -59,7 +59,7 @@ export default class AccountBackendController extends Controller {
        await user.as(UserPasswordComponent).setPassword(req.body.new_password, 'new_password');
        await user.save();

-        req.flash('success', `New password set for ${user.as(UserNameComponent).getName()}`);
+        req.flash('success', `New password set for ${user.as(UserNameComponent).name}`);
        res.redirect(Controller.route('backend-list-users'));
    }
 }
--- a/src/controllers/backend/MailboxBackendController.ts
+++ b/src/controllers/backend/MailboxBackendController.ts
@ -52,9 +52,16 @@ export default class MailboxBackendController extends Controller {
            .get();

        res.render('backend/mailboxes', {
+            users: [{
+                value: 0,
+                display: 'Public',
+            }, ...(await User.select().get()).map(u => ({
+                value: u.id,
+                display: u.name,
+            }))],
            mailboxes: await Promise.all(users.map(async user => ({
                id: user.id,
-                username: user.as(UserNameComponent).getName(),
+                username: user.as(UserNameComponent).name,
                name: await (await user.as(UserMailIdentityComponent).mainMailIdentity.get())?.toEmail(),
                identity_count: (await user.as(UserMailIdentityComponent).mailIdentities.get()).length,
                domain_count: (await user.as(UserMailIdentityComponent).mailDomains.get()).length,
@ -75,7 +82,7 @@ export default class MailboxBackendController extends Controller {
        res.render('backend/mailbox', {
            mailbox: {
                id: user.id,
-                userName: user.as(UserNameComponent).getName(),
+                userName: user.as(UserNameComponent).name,
                name: await mainMailIdentity?.toEmail() || 'Not created.',
                exists: !!mainMailIdentity,
            },
@ -100,16 +107,9 @@ export default class MailboxBackendController extends Controller {
            domains: await Promise.all(mailDomains.map(async domain => ({
                id: domain.id,
                name: domain.name,
-                owner_name: (await domain.owner.get())?.as(UserNameComponent).getName(),
+                owner_name: (await domain.owner.get())?.as(UserNameComponent).name,
                identity_count: (await domain.identities.get()).length,
            }))),
-            users: [{
-                value: 0,
-                display: 'Public',
-            }, ...(await User.select().get()).map(u => ({
-                value: u.id,
-                display: u.name,
-            }))],
        });
    }

--- a/src/models/MailDomain.ts
+++ b/src/models/MailDomain.ts
@ -2,7 +2,6 @@ import Model from "swaf/db/Model";
 import User from "swaf/auth/models/User";
 import {ManyModelRelation, OneModelRelation} from "swaf/db/ModelRelation";
 import MailIdentity from "./MailIdentity";
-import UserNameComponent from "swaf/auth/models/UserNameComponent";

 export default class MailDomain extends Model {
    public id?: number = undefined;
@ -39,6 +38,6 @@ export default class MailDomain extends Model {
    }

    public canCreateAddresses(user: User): boolean {
-        return this.user_id === user.id || this.isPublic() && user.as(UserNameComponent).hasName();
+        return this.user_id === user.id || this.isPublic();
    }
 }
--- a/src/models/UserMailIdentityComponent.ts
+++ b/src/models/UserMailIdentityComponent.ts
@ -3,7 +3,6 @@ import User from "swaf/auth/models/User";
 import {ManyModelRelation, OneModelRelation} from "swaf/db/ModelRelation";
 import MailIdentity from "./MailIdentity";
 import MailDomain from "./MailDomain";
-import config from "config";

 export default class UserMailIdentityComponent extends ModelComponent<User> {
    public main_mail_identity_id?: number = undefined;
@ -28,7 +27,7 @@ export default class UserMailIdentityComponent extends ModelComponent<User> {
        });

    public getMaxPublicAddressesCount(): number {
-        return config.get<number>('mail.max_public_identities_per_user');
+        return 1;
    }

    public async getPublicAddressesCount(): Promise<number> {
--- a/yarn.lock
+++ b/yarn.lock