[Git][NTPsec/ntpsec][master] 3 commits: Update the TODO list.

[Eric S. Raymond]

Eric S. Raymond pushed to branch master at NTPsec / ntpsec


Commits:
5a1f25cd by Eric S. Raymond at 2017-05-26T04:54:13-04:00
Update the TODO list.

- - - - -
9f31a2dc by Eric S. Raymond at 2017-05-26T04:54:13-04:00
Failed unpeer gets error priority.

- - - - -
72dead5a by Eric S. Raymond at 2017-05-26T04:54:13-04:00
Add a known-bug report about silent unpeer failures.

- - - - -


3 changed files:

- devel/TODO
- docs/includes/ntpq-body.txt
- ntpd/ntp_config.c


Changes:

=====================================
devel/TODO
=====================================
--- a/devel/TODO
+++ b/devel/TODO
@@ -48,10 +48,9 @@ of the current startup logic gets discussed.
 
 * A clock driver or auxiliary daemon for PTP.
 
-* The code used for asynchronous DNS lookup is more general, more complex,
-  and more bug-prone than it should be.  Either our version should be
-  rewritten and simplified or (better idea!) it should be replaced by
-  a well-maintained third-party library like c_ares.
+* Invent a syntax for type/unit pairs: use it in the refclock and unpeer
+  directives. Presently it's only possible to unpeer using the deprecated
+  magic address of the driver.
 
 * Hal says "We need a way to inspect/debug bits on the wire. `ntpdate
   -du' is the traditional approach."  Hal's straw man is a new program


=====================================
docs/includes/ntpq-body.txt
=====================================
--- a/docs/includes/ntpq-body.txt
+++ b/docs/includes/ntpq-body.txt
@@ -550,6 +550,11 @@ The command ":config" is no longer accepted; use "config" instead.
 
 == Known Limitations ==
 
+It is possible for a "config unpeer" command to fail silently, yielding
+"Config Succeeded", if it is given a peer identifier that looks like
+a driver type name or a hostname not present in the peer list. The
+error will however be reported in the system log.
+
 include::mrufail.txt[]
 
 You may be able to retrieve partial data in very high-traffic


=====================================
ntpd/ntp_config.c
=====================================
--- a/ntpd/ntp_config.c
+++ b/ntpd/ntp_config.c
@@ -2838,7 +2838,7 @@ config_unpeers(
 
 		/* Misguided attempt to unpeer by driver type name? */
 		if (strchr(name, '.') == NULL && strchr(name, ':') == NULL) {
-		    msyslog(LOG_NOTICE, "refclocks cannot be unpeered by type.");
+		    msyslog(LOG_ERROR, "refclocks cannot be unpeered by type.");
 		    continue;
 		}
 



View it on GitLab: https://gitlab.com/NTPsec/ntpsec/compare/709a3e71f0b152db024b20680da643628935a130...72dead5a7f1ce678a60dd1092092061137f9d21d

---
View it on GitLab: https://gitlab.com/NTPsec/ntpsec/compare/709a3e71f0b152db024b20680da643628935a130...72dead5a7f1ce678a60dd1092092061137f9d21d
You're receiving this email because of your account on gitlab.com.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ntpsec.org/pipermail/vc/attachments/20170526/e76c36fa/attachment.html>