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
URL:
Keywords: patch
Depends on:
Blocks:
 
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+

Attachments
SVN diff (1.06 KB, patch)
2021-04-07 18:29 UTC, Andrej Ebert 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

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 freebsd_triage 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 freebsd_triage 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 freebsd_triage 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!