From 3101bb6ab859c69acf51301cdb988f855c2ccdbc Mon Sep 17 00:00:00 2001 From: Bruno Sutic Date: Tue, 8 Sep 2020 10:30:08 +0200 Subject: [PATCH] Fix async http adapter warning Problem: when using async http adapter rspec outputs the following warning `lib/webmock/http_lib_adapters/async_http_client_adapter.rb:154: warning: instance variable @alpn_protocol not initialized` Reason for that is: `@alpn_protocol` is never defined. Solution: don't use `@alpn_protocol` --- lib/webmock/http_lib_adapters/async_http_client_adapter.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/webmock/http_lib_adapters/async_http_client_adapter.rb b/lib/webmock/http_lib_adapters/async_http_client_adapter.rb index 8b06e1d4d..2bccb1e64 100644 --- a/lib/webmock/http_lib_adapters/async_http_client_adapter.rb +++ b/lib/webmock/http_lib_adapters/async_http_client_adapter.rb @@ -151,7 +151,7 @@ def inspect def create_connected_sockets Async::IO::Socket.pair(Socket::AF_UNIX, Socket::SOCK_STREAM).tap do |sockets| sockets.each do |socket| - socket.instance_variable_set :@alpn_protocol, @alpn_protocol + socket.instance_variable_set :@alpn_protocol, nil socket.instance_eval do def alpn_protocol nil # means HTTP11 will be used for HTTPS