diff options
| author | Craig Jennings <c@cjennings.net> | 2026-08-19 12:32:02 -0700 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-08-19 12:32:02 -0700 |
| commit | 4c1996c8f60ee1d3975f344456ad1e11c758103d (patch) | |
| tree | 87a9c441a5193630f1cd30ed9c5b46d8ffb2d64f /docs/prototypes/2026-07-02-desktop-settings-panel-prototype-26.html | |
| parent | be0a320dd35f4f4a87341bc3f41132a2a83dbcd7 (diff) | |
| download | archsetup-4c1996c8f60ee1d3975f344456ad1e11c758103d.tar.gz archsetup-4c1996c8f60ee1d3975f344456ad1e11c758103d.zip | |
fix(check): read NTP sources only from files chrony is told to read
Check 6 globbed /etc/chrony.d unconditionally. A drop-in is inert unless chrony.conf names its directory, and Arch's stock chrony.conf names none, so a machine with the IP-addressed source on disk and no confdir line would show the literal and pass. That is a false pass on exactly the misconfiguration the check exists to catch, and it describes a file chrony never opens.
Sources now come only from chrony.conf plus whatever confdir or sourcedir it actually names. The config path is a seam so the confdir logic can be tested against a fixture instead of the real /etc.
This should have been in the previous commit, whose message already describes it. I staged before reviewing, fixed the finding, then committed the stale index.
Diffstat (limited to 'docs/prototypes/2026-07-02-desktop-settings-panel-prototype-26.html')
0 files changed, 0 insertions, 0 deletions
