Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Email quota validator error for users #5566

Closed
stephdl opened this issue Aug 14, 2018 · 6 comments
Closed

Email quota validator error for users #5566

stephdl opened this issue Aug 14, 2018 · 6 comments
Labels
verified All test cases were verified successfully
Projects
NethServer 7

Comments

@stephdl
Copy link

stephdl commented Aug 14, 2018 •
edited

Steps to reproduce

  • Install a mail server
  • Try to set a quota for a user superior to 50GB
  • Error from the validator, you cannot set between 60GB to 100GB
  • unlimited is allowed

Expected behavior

I expect the quota validator allows me to set the quota I need

Actual behavior

due to an error in /usr/share/nethesis/NethServer/Module/MailAccount/User/Modify.php, the validator check the value is maximum 50GB or unlimited

Components
nethserver-mail-server & nethserver-mail2-server must be patched

See also

https://community.nethserver.org/t/email-user-quota/10185

thank to netbixAntonio
@stephdl stephdl added the bug A defect of the software label Aug 14, 2018
@stephdl stephdl self-assigned this Aug 14, 2018
@stephdl stephdl added this to ✋ Needs review in NethServer 7 via automation Aug 14, 2018
@stephdl stephdl moved this from ✋ Needs review to ⚙ Developing in NethServer 7 Aug 14, 2018
This was referenced Aug 14, 2018
Merged
Merged
stephdl added a commit to NethServer/nethserver-mail-server that referenced this issue Aug 21, 2018
@stephdl
User quota set to a maximum of 100GB (#60)
1a69064 
NethServer/dev#5566
stephdl added a commit to NethServer/nethserver-mail that referenced this issue Aug 21, 2018
@stephdl
User quota set to a maximum of 100GB (#68)
fbc61e5 
NethServer/dev#5566
@nethbot
Copy link
Member

nethbot commented Aug 21, 2018

in 7.5.1804/testing:

@nethbot
Copy link
Member

nethbot commented Aug 21, 2018

in 7.5.1804/testing:

@stephdl
Copy link
Author

stephdl commented Aug 21, 2018

QA

you must test nethserver-mail-server AND nethserver-mail2-server

  • install the mail server
  • install nethserver-directory (or dc)
  • set a specific email quota for a user, from 50GB to 100GB (test several numbers)
  • you must have no errors

@stephdl stephdl added testing Packages are available from testing repositories and removed bug A defect of the software labels Aug 21, 2018
@stephdl stephdl removed their assignment Aug 21, 2018
@gsanchietti
Copy link
Member

The bug has been resolved.

@gsanchietti gsanchietti added verified All test cases were verified successfully and removed testing Packages are available from testing repositories labels Aug 28, 2018
@nethbot
Copy link
Member

nethbot commented Aug 28, 2018

in 7.5.1804/updates:

@nethbot
Copy link
Member

nethbot commented Aug 28, 2018

in 7.5.1804/updates:

NethServer 7 automation moved this from ⚙ Developing to 🗑 Done Aug 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
verified All test cases were verified successfully
Projects
No open projects
NethServer 7
🗑 Done
Milestone
No milestone
Development

No branches or pull requests
3 participants
@gsanchietti @stephdl @nethbot