From 91f0759631afbc827d8fd7b3cd149ddd21326cb6 Mon Sep 17 00:00:00 2001 From: Yasin Date: Sun, 19 Nov 2017 02:19:00 +0000 Subject: [PATCH 1/2] socket.gethostbyname(socket.gethostname()) will throw an exception if the 'host' can't be resolved either via DNS or via local-hosts file. Setting 'localhost' under such cases. --- cmreslogging/handlers.py | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/cmreslogging/handlers.py b/cmreslogging/handlers.py index 7f6fb01..973b746 100644 --- a/cmreslogging/handlers.py +++ b/cmreslogging/handlers.py @@ -190,8 +190,13 @@ def __init__(self, self.index_name_frequency = index_name_frequency self.es_doc_type = es_doc_type self.es_additional_fields = es_additional_fields.copy() - self.es_additional_fields.update({'host': socket.gethostname(), - 'host_ip': socket.gethostbyname(socket.gethostname())}) + try: + + self.es_additional_fields.update({'host': socket.gethostname(), + 'host_ip': socket.gethostbyname(socket.gethostname())}) + except: + self.es_additional_fields.update({'host': 'localhost', + 'host_ip': '127.0.0.1'}) self.raise_on_indexing_exceptions = raise_on_indexing_exceptions self.default_timestamp_field_name = default_timestamp_field_name From 1539d2c77de70a2d0f4577f26c81973031b6a94a Mon Sep 17 00:00:00 2001 From: yamadmia <33720699+yamadmia@users.noreply.github.com> Date: Sun, 19 Nov 2017 02:36:02 +0000 Subject: [PATCH 2/2] Update handlers.py --- cmreslogging/handlers.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmreslogging/handlers.py b/cmreslogging/handlers.py index 973b746..eb7f694 100644 --- a/cmreslogging/handlers.py +++ b/cmreslogging/handlers.py @@ -194,7 +194,7 @@ def __init__(self, self.es_additional_fields.update({'host': socket.gethostname(), 'host_ip': socket.gethostbyname(socket.gethostname())}) - except: + except Exception: self.es_additional_fields.update({'host': 'localhost', 'host_ip': '127.0.0.1'}) self.raise_on_indexing_exceptions = raise_on_indexing_exceptions