diff options
author | Arne Juul <arnej@yahoo-inc.com> | 2018-05-14 12:54:00 +0000 |
---|---|---|
committer | Arne Juul <arnej@yahoo-inc.com> | 2018-05-15 09:54:08 +0000 |
commit | 27aba1045144a6bc13399f2ee978046989fb1524 (patch) | |
tree | 83b18e6bc431570a408505ed7680853782966eb0 /vespalib | |
parent | 1b23e2014d0402b9d3de71c66304878057aec6e4 (diff) |
cosmetic changes to error messages
* some rewording to make it easier to find the corresponding
documentation.
Diffstat (limited to 'vespalib')
-rw-r--r-- | vespalib/src/apps/vespa-detect-hostname/detect_hostname.cpp | 3 | ||||
-rw-r--r-- | vespalib/src/apps/vespa-validate-hostname/validate_hostname.cpp | 8 |
2 files changed, 6 insertions, 5 deletions
diff --git a/vespalib/src/apps/vespa-detect-hostname/detect_hostname.cpp b/vespalib/src/apps/vespa-detect-hostname/detect_hostname.cpp index a1387c6519d..057d45e8ef5 100644 --- a/vespalib/src/apps/vespa-detect-hostname/detect_hostname.cpp +++ b/vespalib/src/apps/vespa-detect-hostname/detect_hostname.cpp @@ -45,6 +45,7 @@ int main(int, char **) { return 0; } } - fprintf(stderr, "ERROR: failed to detect hostname\n"); + fprintf(stderr, "FATAL: hostname detection failed\n"); + // XXX we should explain why it failed return 1; } diff --git a/vespalib/src/apps/vespa-validate-hostname/validate_hostname.cpp b/vespalib/src/apps/vespa-validate-hostname/validate_hostname.cpp index da4907d4c91..c97927884f8 100644 --- a/vespalib/src/apps/vespa-validate-hostname/validate_hostname.cpp +++ b/vespalib/src/apps/vespa-validate-hostname/validate_hostname.cpp @@ -19,7 +19,7 @@ std::set<vespalib::string> make_ip_set() { vespalib::string normalize(const vespalib::string &hostname) { vespalib::string canon_name = SocketAddress::normalize(hostname); if (canon_name != hostname) { - fprintf(stderr, "warning: host name (%s) is not canonical (canonical host name: %s)\n", + fprintf(stderr, "warning: hostname validation: '%s' is not same as canonical hostname '%s'\n", hostname.c_str(), canon_name.c_str()); } return canon_name; @@ -31,7 +31,7 @@ void check_reverse(const vespalib::string &hostname, const SocketAddress &addr) for (size_t i = 0; !reverse.empty() && (i < 10); ++i) { if (seen.count(reverse) == 0) { seen.insert(reverse); - fprintf(stderr, "warning: conflicting reverse lookup: %s->%s->%s\n", + fprintf(stderr, "warning: hostname validation: found conflicting reverse lookup: '%s' -> %s -> '%s'\n", hostname.c_str(), addr.ip_address().c_str(), reverse.c_str()); } reverse = addr.reverse_lookup(); @@ -53,14 +53,14 @@ int main(int argc, char **argv) { auto addr_list = SocketAddress::resolve(80, hostname.c_str()); if (addr_list.empty()) { valid = false; - fprintf(stderr, "ERROR: host name (%s) could not be resolved\n", + fprintf(stderr, "FATAL: hostname validation failed: '%s' could not be resolved\n", hostname.c_str()); } for (const SocketAddress &addr: addr_list) { vespalib::string ip_addr = addr.ip_address(); if (my_ip_set.count(ip_addr) == 0) { valid = false; - fprintf(stderr, "ERROR: host name (%s) resolves to ip address not owned by this host (%s)\n", + fprintf(stderr, "FATAL: hostname validation failed: '%s' resolves to ip address not owned by this host (%s)\n", hostname.c_str(), ip_addr.c_str()); } else { check_reverse(hostname, addr); |