On Thu, Apr 02, 2015 at 07:31:52AM -0400, Bruce Momjian wrote: > On Thu, Apr 2, 2015 at 01:44:59AM -0400, Noah Misch wrote: > > On Wed, Apr 01, 2015 at 10:49:01PM -0400, Bruce Momjian wrote: > > > On Sat, Jan 10, 2015 at 02:53:13PM -0500, Tom Lane wrote: > > > > While looking at fe-auth.c I noticed quite a few places that weren't > > > > bothering to make error messages localizable (ie, missing libpq_gettext > > > > calls), and/or were failing to add a trailing newline as expected in > > > > libpq error messages. Perhaps these are intentional but I doubt it. > > > > Most of the instances seemed to be SSPI-related. > > > > > > > > I have no intention of fixing these myself, but whoever committed that > > > > code should take a second look. > > > > > > I looked through that file and only found two cases; patch attached. > > > > Tom mentioned newline omissions, which you'll find in pg_SSPI_error(). > > Oh, I accidentally saw the backend version of that function, which > looked fine. I have attached a patch for that.
That patch looks reasonable. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
