Skip to content

Commit e18aa61

Browse files
authored
Merge pull request #197 from headius/no_double_interrupt_test_server
Don't double-interrupt the test HTTP server
2 parents 0b60ca6 + 54025b3 commit e18aa61

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

test/net/http/utils.rb

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,12 +27,14 @@ def initialize(config, &block)
2727
def start
2828
@thread = Thread.new do
2929
loop do
30-
socket = @ssl_server ? @ssl_server.accept : @server.accept
30+
socket = (@ssl_server || @server).accept
3131
run(socket)
3232
rescue
3333
ensure
34-
socket.close if socket
34+
socket&.close
3535
end
36+
ensure
37+
(@ssl_server || @server).close
3638
end
3739
end
3840

@@ -42,7 +44,6 @@ def run(socket)
4244

4345
def shutdown
4446
@thread&.kill
45-
@server&.close
4647
@thread&.join
4748
end
4849

0 commit comments

Comments
 (0)