Re: [PATCH] Make configuration file "include" directive handling more robust

2019-08-27 Thread Ian Barwick
On 8/25/19 4:39 AM, Tom Lane wrote: Ian Barwick writes: On 7/17/19 5:34 PM, Kyotaro Horiguchi wrote:> Hello. I don't think this is new to 12. No, though I'm not sure how much this would be seen as a bugfix and how far back it would be sensible to patch. I think this is worth considering a

Re: [PATCH] Make configuration file "include" directive handling more robust

2019-08-24 Thread Tom Lane
Ian Barwick writes: > On 7/17/19 5:34 PM, Kyotaro Horiguchi wrote:> Hello. >>> I don't think this is new to 12. > No, though I'm not sure how much this would be seen as a bugfix > and how far back it would be sensible to patch. I think this is worth considering as a bugfix; although I'm afraid w

Re: [PATCH] Make configuration file "include" directive handling more robust

2019-07-17 Thread Ian Barwick
On 7/17/19 5:34 PM, Kyotaro Horiguchi wrote:> Hello. > > At Wed, 17 Jul 2019 12:29:43 +0900, Ian Barwick wrote in <8c8bcbca-3bd9-dc6e-8986-04a5abdef...@2ndquadrant.com> >> Hi >> >> While poking about with [1], I noticed a few potential issues with the >> inclusion handling for configuration file

Re: [PATCH] Make configuration file "include" directive handling more robust

2019-07-17 Thread Kyotaro Horiguchi
Hello. At Wed, 17 Jul 2019 12:29:43 +0900, Ian Barwick wrote in <8c8bcbca-3bd9-dc6e-8986-04a5abdef...@2ndquadrant.com> > Hi > > While poking about with [1], I noticed a few potential issues with the > inclusion handling for configuration files; another issue is > demonstrated in [2]. > > [1] >

[PATCH] Make configuration file "include" directive handling more robust

2019-07-16 Thread Ian Barwick
Hi While poking about with [1], I noticed a few potential issues with the inclusion handling for configuration files; another issue is demonstrated in [2]. [1] https://www.postgresql.org/message-id/aed6cc9f-98f3-2693-ac81-52bb0052307e%402ndquadrant.com ("Stop ALTER SYSTEM from making bad as