starred/ViMbAdmin-opensolutions
1
0
Fork 0
mirror of https://github.com/opensolutions/ViMbAdmin.git synced 2026-04-26 16:55:59 +03:00
Code Issues 16 Projects Releases 10 Packages Wiki Activity

[GH-ISSUE #289] Database column mailbox.username is not unique #237

New issue
Closed
opened 2026-02-26 09:36:58 +03:00 by kerem · 1 comment
kerem commented 2026-02-26 09:36:58 +03:00
Owner
Copy link

Originally created by @PhrozenByte on GitHub (Jul 13, 2022).
Original GitHub issue: https://github.com/opensolutions/ViMbAdmin/issues/289

mailbox.username currently isn't required to be unique in the database (uniqueness is currently only enforced by code), nor is the column even indexed. Querying the ViMbAdmin database using mailbox.username as (supposed to be unique) identifier is one of the main database queries any mail server software (Dovecot, Postfix, etc.) performs frequently (up to thousands of queries a day, depending on the number of users). mailbox.local_part should be indexed, too.

Originally created by @PhrozenByte on GitHub (Jul 13, 2022). Original GitHub issue: https://github.com/opensolutions/ViMbAdmin/issues/289 `mailbox`.`username` currently isn't required to be unique in the database (uniqueness is currently only [enforced by code](https://github.com/opensolutions/ViMbAdmin/blob/6fc7b6a7be3eb9264b27caf4e48d8c90bafc3ce9/application/controllers/MailboxController.php#L282-L290)), nor is the column even indexed. Querying the ViMbAdmin database using `mailbox`.`username` as (supposed to be unique) identifier is one of the main database queries any mail server software (Dovecot, Postfix, etc.) performs frequently (up to thousands of queries a day, depending on the number of users). `mailbox`.`local_part` should be indexed, too.
kerem closed this issue 2026-02-26 09:36:59 +03:00
kerem commented 2026-02-26 09:37:00 +03:00
Author
Owner
Copy link

@barryo commented on GitHub (Feb 23, 2023):

Good catch. Fixed in 5836ad571a and can be updated via:

# sanity check the output of:
./bin/doctrine2-cli.php orm:schema-tool:update --dump-sql

# and then run:
./bin/doctrine2-cli.php orm:schema-tool:update --force
<!-- gh-comment-id:1442507622 --> @barryo commented on GitHub (Feb 23, 2023): Good catch. Fixed in 5836ad571a5017351605a341c1a3c5c02e348e3c and can be updated via: ``` # sanity check the output of: ./bin/doctrine2-cli.php orm:schema-tool:update --dump-sql # and then run: ./bin/doctrine2-cli.php orm:schema-tool:update --force ```
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
dependabot/composer/smarty/smarty-4.3.1
v2
3.4.1
3.4.0
3.3.0
3.2.1
3.2.0
3.1.1
3.1.0
3.0.15
3.0.14
3.0.13
3.0.12
3.0.11
2.2.4
3.0.10
3.0.9
3.0.8
3.0.7
3.0.6
3.0.5
3.0.4
3.0.3
3.0.2
3.0.1
3.0.0
2.2.3
2.2.2
2.2.1
2.2.0
2.1.0
2.0.8
2.0.7
2.0.6
2.0.5
2.0.4
2.0.3
2.0.2
2.0.1
2.0.0
0.3.4
0.3.3
0.3.2
0.3.1
0.3
Labels
Clear labels   
bug

   
feature

   
feature

   
improvement

   
improvement

   
pull-request

Mirrored from GitHub Pull Request

No labels
bug
feature
feature
improvement
improvement
pull-request
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/ViMbAdmin-opensolutions#237
No description provided.