nextcloud / nextcloud/mail

Enable option to test mail speed against self configured mail server speed setting

Open
#12,958 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

1. to develop enhancement
Dominant language
JavaScript
Stars
1k
Forks
348
Avg merge
12h 28m
Merged PRs (30d)
91

Description

Is your feature request related to a problem? Please describe.

In merge https://github.com/nextcloud/mail/pull/10659 a test to check mail server performance was implemented. The test checks for speed of mail server connection performance. Two tests are performed, and both of these tests have hard coded settings of 1s being the maximum time allowable for the test to be completed. The permanent figure of 1s should be adjustable by the server admin as they see fit for their own systems.

Describe the solution you'd like

Add a setting to the admin groupware page for the mail app to enable admins to adjust the time setting for comparing mail performance by server

Describe alternatives you've considered

I have manually modified lib/SetupChecks/MailConnectionPerformance.php as a temporary measure

Additional context

No response

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with lib/SetupChecks/MailConnectionPerformance.php and trace the two checks that currently use the one-second limit. Then locate the mail app's admin groupware settings and connect the configurable value to those checks; done means admins can adjust the threshold and both performance comparisons use it.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend, frontend, performance
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.