Bug 254859 - net/chrony: Add --enable-ntp-signd option
Summary: net/chrony: Add --enable-ntp-signd option
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Matthias Andree
Keywords: patch
Depends on:
Reported: 2021-04-07 18:29 UTC by Andrej Ebert
Modified: 2021-04-07 21:00 UTC (History)
1 user (show)

See Also:
mandree: maintainer-feedback+

SVN diff (1.06 KB, patch)
2021-04-07 18:29 UTC, Andrej Ebert
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andrej Ebert 2021-04-07 18:29:30 UTC
Created attachment 223902 [details]
SVN diff

Since this is needed to provide time to Windows domain clients when running a Samba 4 AD DC, I propose to add --enable-ntp-signd as an option. I didn't set it as default, but arguably it would be of benefit for many users (I looked a long time to find out why my windows clients just wouldn't sync time....). Many Linux distributions seem to set it in their packages.
Comment 1 Matthias Andree freebsd_committer 2021-04-07 18:34:50 UTC
Hi Andrej,

thanks for the contribution. Does this have any special requirements? If no, I am inclined to just enable it by default and not make it an option.
Comment 2 Matthias Andree freebsd_committer 2021-04-07 18:59:58 UTC
I have committed this. Please test chrony 4.0_1 and report back if that works for you.
Comment 3 commit-hook freebsd_committer 2021-04-07 19:00:03 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=11ff2d7e81002e788f8e54ad7b60daba43db8e50

commit 11ff2d7e81002e788f8e54ad7b60daba43db8e50
Author:     Matthias Andree <mandree@FreeBSD.org>
AuthorDate: 2021-04-07 18:57:26 +0000
Commit:     Matthias Andree <mandree@FreeBSD.org>
CommitDate: 2021-04-07 18:59:33 +0000

    net/chrony: enable MS-SNTP authentication (SIGND)
    PR:             254859
    Reported by:    Andrej Ebert

 net/chrony/Makefile | 2 ++
 1 file changed, 2 insertions(+)
Comment 4 Andrej Ebert 2021-04-07 20:18:51 UTC
(In reply to Matthias Andree from comment #2)

Works for me, thanks for the quick commit!