Re: encoding of PostgreSQL messages - Mailing list pgsql-general

From Hiroshi Inoue
Subject Re: encoding of PostgreSQL messages
Date
Msg-id 49831889.2020103@tpf.co.jp
Whole thread Raw
In response to Re: encoding of PostgreSQL messages  (Bruce Momjian <bruce@momjian.us>)
Responses Re: encoding of PostgreSQL messages  (Bruce Momjian <bruce@momjian.us>)
List pgsql-general
Hi,

This topic seems to be related to the bug report
    [ODBC] Localized error messages, wrong charset
.

Bruce Momjian wrote:
> Added to TODO:
>
>     Improve encoding of connection startup messages sent to the client
>
>         Currently some authentication error messages are sent in the server
>         encoding

It it true ?
IIRC the backend knows nothing about the server encoding in
authentication phase.

Psqlodbc Unicode driver sends connection startup message which
contains the client_encoding(=UTF8) guc parameter. Attached is
  a trial patch so that the psqlodbc Unicode driver can get
properly localized password error messages.

regards,
Hiroshi Inoue
Index: postmaster/postmaster.c
===================================================================
RCS file: /projects/cvsroot/pgsql/src/backend/postmaster/postmaster.c,v
retrieving revision 1.570
diff -c -c -r1.570 postmaster.c
*** postmaster/postmaster.c    4 Jan 2009 22:19:59 -0000    1.570
--- postmaster/postmaster.c    30 Jan 2009 14:05:35 -0000
***************
*** 1552,1557 ****
--- 1552,1560 ----
                                              pstrdup(nameptr));
                  port->guc_options = lappend(port->guc_options,
                                              pstrdup(valptr));
+                 if (stricmp(nameptr, "client_encoding") == 0 &&
+                     stricmp(valptr, "UTF8") == 0)
+                     bind_textdomain_codeset(PG_TEXTDOMAIN("postgres"), "UTF-8");
              }
              offset = valoffset + strlen(valptr) + 1;
          }

pgsql-general by date:

Previous
From: A B
Date:
Subject: Re: Runaway postgres process?
Next
From: Asko Oja
Date:
Subject: Re: database/table snapshot