pg 1.1.2-x64-mingw32 → 1.1.3-x64-mingw32

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 297b04be76576c28a27569752072310064ffa03e
-  data.tar.gz: 622e7a4704c211a83f75d4ee1df21c1babb576c0
+  metadata.gz: f6ceb296d0fc4b59f0ec35a39f733f945737d512
+  data.tar.gz: 5dddb54d70cddd40ad0e06c2d4833d6967e9355d
 SHA512:
-  metadata.gz: d1d8fe76d249606aab43bd746f5e46f894a136f8717d72146fb7f3a1eb5c75d3938b31a41f0ad387a5160f76965233b18272d4785986fa5c7a9988f1e6854180
-  data.tar.gz: 102409058cf84ee4b01bd94d04c142a24b71d7a2f9d6e19ed47776b959bd4b767a4e9e66d4d7e8ede7f7ff0b7c82a7de3e0662fdd5c45082f3314c96f4f44877
+  metadata.gz: dcffbd9981e91a73e9bcd1ee5e49aa0587da99271defdfb55d741368e82ec61caac5f6e150dc0c9f9e191cef11a7479ce4e2669375aad5436326ae2c5106d14c
+  data.tar.gz: d0ae758db5ed159c5771746a6f50bc87bd978bba9df10a908c2171e1fa0ca08ee6310f5a1b4ec79f78c8d66b40d1143ff923ce357b097b05a1e5bfd101831368

data.tar.gz.sig

@@ -1,2 +1,4 @@
-FE�,�0���j	�=�W�`����F=�y	��f�暥�;�)��"=�j冫����Y;�qٳH�%��؛����q)�]S=�6��������;l@c��y�al�(�X�[jм�X��kY��,<�ߟj���T�u�;ۅ��X��,��W���7P����6����H�����~�nō���O}�T�5T"-6��z����E�΍n��0�
�y�
�v�͌��QEvMUǜ��9Ȫ�e�fFr�S&Jf���C�8��
-�;�?�jCb���hu�~��DZ
���2�Tv/���Y�z>>a���-����`]m��0>;H��3������j	MՏ�[
+(^�s���M���;ہ7�V��g<:��r�t!�5.l�]������f��Wf<�Sm9;pd���H�~
+��2l{�$U��F�1�7E����I��>�ة����r���IvB%��	�=xj\.oB�4����+�.���ꀷ�8T��E��|�I{Mߌ��,�\?s����VO�!��g�߿���ڋ�M���Pӹ�����((ʏ%����~�ֶ��
+��X�iR�vO�4z�
+s���u��li��̼�1 (��º'��{@3�^�m�`�z���G�c�m)d4'�iQN>|�/lBi�%	���`���a�n|*��S9
ej�B���&	�cY�yd�9���y

data/History.rdoc

@@ -1,3 +1,8 @@
+== v1.1.3 [2018-09-06] Michael Granger <ged@FaerieMUD.org>
+
+- Revert opimization that was sometimes causing EBADF in rb_wait_for_single_fd().
+
+
 == v1.1.2 [2018-08-28] Michael Granger <ged@FaerieMUD.org>
 
 - Don't generate aliases for JOHAB encoding.

data/ext/pg.h

@@ -101,9 +101,6 @@ typedef struct {
 	/* Kind of PG::Coder object for casting COPY rows to ruby values */
 	VALUE decoder_for_get_copy_data;
 
-	/* The connection socket, used for rb_wait_for_single_fd() */
-	int socket;
-
 	/* enable/disable guessing size of PGresult's allocated memory */
 	int guess_result_memsize;
 } t_pg_connection;

data/ext/pg_connection.c

@@ -1,6 +1,6 @@
 /*
  * pg_connection.c - PG::Connection class extension
- * $Id: pg_connection.c,v be48d118eeed 2018/08/25 11:35:03 lars $
+ * $Id: pg_connection.c,v b49f54dc755b 2018/09/01 12:52:41 lars $
  *
  */
 
@@ -198,7 +198,6 @@ pgconn_s_allocate( VALUE klass )
 	this->decoder_for_get_copy_data = Qnil;
 	this->trace_stream = Qnil;
 	this->external_encoding = Qnil;
-	this->socket = -1;
 	this->guess_result_memsize = 1;
 
 	return self;
@@ -272,6 +271,7 @@ pgconn_init(int argc, VALUE *argv, VALUE self)
 	this = pg_get_connection( self );
 	conninfo = rb_funcall2( rb_cPGconn, rb_intern("parse_connect_args"), argc, argv );
 	this->pgconn = gvl_PQconnectdb(StringValueCStr(conninfo));
+
 	if(this->pgconn == NULL)
 		rb_raise(rb_ePGerror, "PQconnectdb() unable to allocate structure");
 
@@ -281,10 +281,6 @@ pgconn_init(int argc, VALUE *argv, VALUE self)
 		rb_exc_raise(error);
 	}
 
-	this->socket = PQsocket( this->pgconn );
-	if ( this->socket < 0 )
-		rb_raise(rb_eConnectionBad, "PQsocket() can't get socket descriptor");
-
 	pgconn_set_default_encoding( self );
 
 	if (rb_block_given_p()) {
@@ -335,10 +331,6 @@ pgconn_s_connect_start( int argc, VALUE *argv, VALUE klass )
 		rb_exc_raise(error);
 	}
 
-	this->socket = PQsocket( this->pgconn );
-	if ( this->socket < 0 )
-		rb_raise(rb_eConnectionBad, "PQsocket() can't get socket descriptor");
-
 	if ( rb_block_given_p() ) {
 		return rb_ensure( rb_yield, rb_conn, pgconn_finish, rb_conn );
 	}
@@ -2337,15 +2329,18 @@ pgconn_notifies(VALUE self)
 int rb_w32_wait_events( HANDLE *events, int num, DWORD timeout );
 
 static void *
-wait_socket_readable( t_pg_connection *this, struct timeval *ptimeout, void *(*is_readable)(PGconn *) )
+wait_socket_readable( PGconn *conn, struct timeval *ptimeout, void *(*is_readable)(PGconn *) )
 {
-	PGconn *conn = this->pgconn;
+	int sd = PQsocket( conn );
 	void *retval;
 	struct timeval aborttime={0,0}, currtime, waittime;
 	DWORD timeout_milisec = INFINITE;
 	DWORD wait_ret;
 	WSAEVENT hEvent;
 
+	if ( sd < 0 )
+		rb_raise(rb_eConnectionBad, "PQsocket() can't get socket descriptor");
+
 	hEvent = WSACreateEvent();
 
 	/* Check for connection errors (PQisBusy is true on connection errors) */
@@ -2360,7 +2355,7 @@ wait_socket_readable( t_pg_connection *this, struct timeval *ptimeout, void *(*i
 	}
 
 	while ( !(retval=is_readable(conn)) ) {
-		if ( WSAEventSelect(this->socket, hEvent, FD_READ|FD_CLOSE) == SOCKET_ERROR ) {
+		if ( WSAEventSelect(sd, hEvent, FD_READ|FD_CLOSE) == SOCKET_ERROR ) {
 			WSACloseEvent( hEvent );
 			rb_raise( rb_eConnectionBad, "WSAEventSelect socket error: %d", WSAGetLastError() );
 		}
@@ -2412,13 +2407,16 @@ wait_socket_readable( t_pg_connection *this, struct timeval *ptimeout, void *(*i
 /* non Win32 */
 
 static void *
-wait_socket_readable( t_pg_connection *this, struct timeval *ptimeout, void *(*is_readable)(PGconn *))
+wait_socket_readable( PGconn *conn, struct timeval *ptimeout, void *(*is_readable)(PGconn *))
 {
-	PGconn *conn = this->pgconn;
+	int sd = PQsocket( conn );
 	int ret;
 	void *retval;
 	struct timeval aborttime={0,0}, currtime, waittime;
 
+	if ( sd < 0 )
+		rb_raise(rb_eConnectionBad, "PQsocket() can't get socket descriptor");
+
 	/* Check for connection errors (PQisBusy is true on connection errors) */
 	if ( PQconsumeInput(conn) == 0 )
 		rb_raise( rb_eConnectionBad, "PQconsumeInput() %s", PQerrorMessage(conn) );
@@ -2437,7 +2435,7 @@ wait_socket_readable( t_pg_connection *this, struct timeval *ptimeout, void *(*i
 		/* Is the given timeout valid? */
 		if( !ptimeout || (waittime.tv_sec >= 0 && waittime.tv_usec >= 0) ){
 			/* Wait for the socket to become readable before checking again */
-			ret = rb_wait_for_single_fd( this->socket, RB_WAITFD_IN, ptimeout ? &waittime : NULL );
+			ret = rb_wait_for_single_fd( sd, RB_WAITFD_IN, ptimeout ? &waittime : NULL );
 		} else {
 			ret = 0;
 		}
@@ -2484,7 +2482,7 @@ notify_readable(PGconn *conn)
 static VALUE
 pgconn_wait_for_notify(int argc, VALUE *argv, VALUE self)
 {
-	t_pg_connection *this = pg_get_connection_safe( self );
+	PGconn *conn = pg_get_pgconn( self );
 	PGnotify *pnotification;
 	struct timeval timeout;
 	struct timeval *ptimeout = NULL;
@@ -2500,7 +2498,7 @@ pgconn_wait_for_notify(int argc, VALUE *argv, VALUE self)
 		ptimeout = &timeout;
 	}
 
-	pnotification = (PGnotify*) wait_socket_readable( this, ptimeout, notify_readable);
+	pnotification = (PGnotify*) wait_socket_readable( conn, ptimeout, notify_readable);
 
 	/* Return nil if the select timed out */
 	if ( !pnotification ) return Qnil;
@@ -3078,7 +3076,7 @@ get_result_readable(PGconn *conn)
  */
 static VALUE
 pgconn_block( int argc, VALUE *argv, VALUE self ) {
-	t_pg_connection *this = pg_get_connection_safe( self );
+	PGconn *conn = pg_get_pgconn( self );
 
 	struct timeval timeout;
 	struct timeval *ptimeout = NULL;
@@ -3093,7 +3091,7 @@ pgconn_block( int argc, VALUE *argv, VALUE self ) {
 		ptimeout = &timeout;
 	}
 
-	ret = wait_socket_readable( this, ptimeout, get_result_readable);
+	ret = wait_socket_readable( conn, ptimeout, get_result_readable);
 
 	if( !ret )
 		return Qfalse;

data/lib/pg.rb

@@ -35,10 +35,10 @@ end
 module PG
 
 	# Library version
-	VERSION = '1.1.2'
+	VERSION = '1.1.3'
 
 	# VCS revision
-	REVISION = %q$Revision: bbf57bf7e583 $
+	REVISION = %q$Revision: 6f611e78845a $
 
 	class NotAllCopyDataRetrieved < PG::Error
 	end

data/spec/pg/connection_spec.rb

@@ -228,6 +228,22 @@ describe PG::Connection do
 
 			res = @conn2.query("SELECT 4")
 		end
+
+		it "can use conn.reset_start to restart the connection" do
+			ios = IO.pipe
+			conn = described_class.connect_start( @conninfo )
+			wait_for_polling_ok(conn)
+
+			# Close the two pipe file descriptors, so that the file descriptor of
+			# newly established connection is probably distinct from the previous one.
+			ios.each(&:close)
+			conn.reset_start
+			wait_for_polling_ok(conn)
+
+			# The new connection should work even when the file descriptor has changed.
+			expect( conn.exec("SELECT 1").values ).to eq([["1"]])
+			conn.close
+		end
 	end
 
 	it "raises proper error when sending fails" do
@@ -914,6 +930,20 @@ describe PG::Connection do
 		expect { conn.finish }.to raise_error( PG::ConnectionBad, /connection is closed/i )
 	end
 
+	it "can use conn.reset to restart the connection" do
+		ios = IO.pipe
+		conn = PG.connect( @conninfo )
+
+		# Close the two pipe file descriptors, so that the file descriptor of
+		# newly established connection is probably distinct from the previous one.
+		ios.each(&:close)
+		conn.reset
+
+		# The new connection should work even when the file descriptor has changed.
+		expect( conn.exec("SELECT 1").values ).to eq([["1"]])
+		conn.close
+	end
+
 	it "closes the IO fetched from #socket_io when the connection is closed", :without_transaction, :socket_io do
 		conn = PG.connect( @conninfo )
 		io = conn.socket_io
@@ -939,7 +969,7 @@ describe PG::Connection do
 		end
 		serv.close
 		expect{ conn.block }.to raise_error(PG::ConnectionBad, /server closed the connection unexpectedly/)
-		expect{ conn.block }.to raise_error(PG::ConnectionBad, /connection not open/)
+		expect{ conn.block }.to raise_error(PG::ConnectionBad, /can't get socket descriptor/)
 	end
 
 	it "sets the fallback_application_name on new connections" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pg
 version: !ruby/object:Gem::Version
-  version: 1.1.2
+  version: 1.1.3
 platform: x64-mingw32
 authors:
 - Michael Granger
@@ -36,7 +36,7 @@ cert_chain:
   X0qdrKi+2aZZ0NGuFj9AItBsVmAvkBGIpX4TEKQp5haEbPpmaqO5nIIhV26PXmyT
   OMKv6pWsoS81vw5KAGBmfX8nht/Py90DQrbRvakATGI=
   -----END CERTIFICATE-----
-date: 2018-08-29 00:00:00.000000000 Z
+date: 2018-09-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hoe-mercurial