zuul
/
zuul
Code Issues Proposed changes

Merge "Fix secondary exception in fingergw"

This commit is contained in:
Zuul 2019-01-23 03:08:54 +00:00 committed by Gerrit Code Review
parent ac66685584 3dc386626b
commit 25f4f7a45c
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
zuul/lib/fingergw.py
View File

@ -64,6 +64,8 @@ class RequestHandler(streamer_utils.BaseFingerRequestHandler):
This method is called by the socketserver framework to handle an
incoming request.
'''
server = None
port = None
try:
build_uuid = self.getCommand()
port_location = self.rpc.get_job_log_stream_address(build_uuid)
@ -73,17 +75,15 @@ class RequestHandler(streamer_utils.BaseFingerRequestHandler):
self.request.sendall(msg.encode('utf-8'))
return
self._fingerClient(
port_location['server'],
port_location['port'],
build_uuid,
)
server = port_location['server']
port = port_location['port']
self._fingerClient(server, port, build_uuid)
except BrokenPipeError: # Client disconnect
return
except Exception:
self.log.exception(
'Finger request handling exception (%s:%s):',
port_location['server'], port_location['port'])
server, port)
msg = 'Internal streaming error'
self.request.sendall(msg.encode('utf-8'))
return