Should perform sanity checks for "override" option

[jneo8]

Currently, "override" option allows one to change "rotate" and "interval" directives to any value, it's not desired.

One could easily set "rotate" and "interval" to invalid value such as "rotate = -5" or "interval = decade" without problems. However, if one want to change them back to original value, then it's impossible. This is mainly because the charm uses regex to change these values [1,2].

/var/log/apt/term.log {
  rotate -5
  decade
  compress
  missingok
  notifempty
    rotate 5
}
/var/log/apt/history.log {
  rotate -5
  decade
  compress
  missingok
  notifempty
    rotate 5
}

[1] https://git.launchpad.net/charm-logrotated/tree/src/lib/lib_logrotate.py#n17 [2] https://git.launchpad.net/charm-logrotated/tree/src/lib/lib_logrotate.py#n90

---

Imported from Launchpad using lp2gh.

• date created: 2023-05-04T11:17:04Z

• owner: raychan96

• assignee: None

• the launchpad url