From: | Noah Misch <noah(at)leadboat(dot)com> |
---|---|
To: | Tatsuo Ishii <ishii(at)postgresql(dot)org> |
Cc: | tgl(at)sss(dot)pgh(dot)pa(dot)us, robertmhaas(at)gmail(dot)com, kondo(at)sraoss(dot)co(dot)jp, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Re: [BUGS] BUG #13611: test_postmaster_connection failed (Windows, listen_addresses = '0.0.0.0' or '::') |
Date: | 2015-10-29 03:37:48 |
Message-ID: | 20151029033748.GB770464@tornado.leadboat.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs pgsql-hackers |
On Tue, Oct 27, 2015 at 05:31:25PM +0900, Tatsuo Ishii wrote:
> > No, PQping("host='127.0.0.1'") fails to reach a listen_addresses='::' server
> > on many systems. Here's what I thought Kondo was proposing:
> >
> > --- a/src/bin/pg_ctl/pg_ctl.c
> > +++ b/src/bin/pg_ctl/pg_ctl.c
> > @@ -649,5 +649,9 @@ test_postmaster_connection(pgpid_t pm_pid, bool do_checkpoint)
> >
> > - /* If postmaster is listening on "*", use localhost */
> > + /* explanation here */
> > if (strcmp(host_str, "*") == 0)
> > strcpy(host_str, "localhost");
> > + else if (strcmp(host_str, "0.0.0.0") == 0)
> > + strcpy(host_str, "127.0.0.1");
> > + else if (strcmp(host_str, "::") == 0)
> > + strcpy(host_str, "::1");
> >
>
> I see. Would you like to commit this?
I am happy to finish it, but I am no less happy if you finish it. Which do
you prefer?
Should the back-branch commits mirror the master branch? A more-cautious
alternative would be to, in back branches, wrap the change in #ifdefs so it
takes effect only on Windows, OpenBSD and NetBSD. It could break setups with
local firewall rules that block connections to "127.0.0.1" or "::1" without
blocking "0.0.0.0" or "::". Such firewall rules sound outlandish enough that
I would be fairly comfortable not worrying about this and making the change
unconditional in all branches. It's a judgment call, though.
From | Date | Subject | |
---|---|---|---|
Next Message | Tatsuo Ishii | 2015-10-29 08:39:36 | Re: Re: [BUGS] BUG #13611: test_postmaster_connection failed (Windows, listen_addresses = '0.0.0.0' or '::') |
Previous Message | Haribabu Kommi | 2015-10-29 00:13:00 | Re: BUG #13741: vacuumdb does not accept valid password |
From | Date | Subject | |
---|---|---|---|
Next Message | Haribabu Kommi | 2015-10-29 05:02:08 | security_barrier view option type mistake in create view document |
Previous Message | Tom Lane | 2015-10-29 03:27:49 | Re: Small improvement to tlist_matches_tupdesc() |