Skip to content

Commit

Permalink
Merge pull request #38 from pmcollins/stop-http
Browse files Browse the repository at this point in the history
Shutdown http server
  • Loading branch information
pmcollins authored Dec 9, 2024
2 parents d6a2daf + 3350c22 commit 51851ed
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
6 changes: 4 additions & 2 deletions src/oteltest/sink/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,7 @@ def stop(self):
class HttpSink:

def __init__(self, listener, logger: logging.Logger, port=4318, daemon=True):
self.httpd = None
self.listener = listener
self.logger = logger
self.port = port
Expand Down Expand Up @@ -133,8 +134,8 @@ def do_POST(this):
this.wfile.write("OK".encode("utf-8"))

# noinspection PyTypeChecker
httpd = HTTPServer(("", self.port), Handler)
httpd.serve_forever()
self.httpd = HTTPServer(("", self.port), Handler)
self.httpd.serve_forever()

def handle_trace(self, post_data, headers):
req = ExportTraceServiceRequest()
Expand All @@ -152,6 +153,7 @@ def handle_logs(self, post_data, headers):
self.listener.handle_logs(req, headers)

def stop(self):
self.httpd.shutdown()
self.svr_thread.join()


Expand Down
2 changes: 1 addition & 1 deletion src/oteltest/version.py
Original file line number Diff line number Diff line change
@@ -1 +1 @@
__version__ = "0.36.0"
__version__ = "0.37.0"

0 comments on commit 51851ed

Please sign in to comment.