Skip to content
Snippets Groups Projects

Check server config section

Merged Jesse Norell requested to merge jnorell/ispconfig3:check_server_config_section into stable-3.1

This checks that a server config section is not empty when saving.

Note that I don't have any any sections in my server config which are currently empty, but if that is valid (ie. a section [header] is allowed with no settings under it), this should be modified or not accepted.

I did see one empty [jailkit] section get saved in datalog records, do to other bug(s)/issue(s), and it came to mind that this could easily be prevented. I don't know exactly how that other issue happened though so I can't test it in a live "bug scenario".

Edited by Jesse Norell

Merge request reports

Merge request pipeline #3251 passed

Merge request pipeline passed for dfd82421

Approval is optional

Merged by Till BrehmTill Brehm 4 years ago (Jul 1, 2020 8:55am UTC)

Merge details

  • Changes merged into stable-3.1 with 132706b8.
  • Deleted the source branch.

Pipeline #3255 passed

Pipeline passed for 132706b8 on stable-3.1

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading