pg 1.1.2-x86-mingw32 → 1.1.3-x86-mingw32

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2e0df91178293f0f6248e8c750999ce96e667ffa
-  data.tar.gz: 2402e6e9de0141349ef98604f6eed4eb1a9f331d
+  metadata.gz: 0c86ddfcf7b79e6cf6d872926775db949011d45f
+  data.tar.gz: 05b496d93f220caf5426fae5457e7921d0e0315c
 SHA512:
-  metadata.gz: f872b0dd91fa0bb7f7d43836a78d1657b4e1b0d0f1f7d5ad8bb94dc1ad188891addb56af800867b06cabea6c720d85597ea2b4a66e87c62a03af495aab7707e3
-  data.tar.gz: d85f5391ecf3590b59bf81863cefa2b7ff6c90812c6bafd9ba372fb8660b4692ec430f178fa2e127b3d5bc99eb06dd1e529697e2e62fe7fe48157afdf2c42b99
+  metadata.gz: 7fc423e112b1867180daf407b9c00a0131b5f57dc4bfa8496e6f5390f5444de160625a82d6f7f6c63cc1a43ad98e973db73ffa625b17861ff0bacaa3f54e7870
+  data.tar.gz: ab17b5437b6bf1802b80dadaa8be8c3ac29f26de41a8889d6b5f1fd3672dc83dd69e5310264a311fa44da627fb083ce840a1eda5b75c8d709da1e108b749a9b5

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: x86-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