pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Michael Paquier <michael@paquier.xyz>
To: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
Cc: Jacob Champion <pchampion@vmware.com>
Cc: pgsql-hackers@postgresql.org <pgsql-hackers@postgresql.org>
Subject: Re: Proposal: Save user's original authenticated identity for logging
Date: Wed, 22 Sep 2021 16:39:10 +0900
Message-ID: <YUrdnoEc4NPfBZfI@paquier.xyz> (raw)
In-Reply-To: <ae1143fb-455c-c80f-ed66-78d45bd93303@enterprisedb.com>
References: <4a87886f034c713c056c43850f55e6e0ba50d8b5.camel@vmware.com>
	<YGJ3FoOfgSao9Cv8@paquier.xyz>
	<e0f0484a1815b26bb99ef9ddc7a110dfd6425931.camel@vmware.com>
	<a4fddbcd2f99c293991b7669bd0da3e5a7d3f93b.camel@vmware.com>
	<YGQk74rdCdb8F9gZ@paquier.xyz>
	<YGUgHMF9zxNvY323@paquier.xyz>
	<d17b919e27474abfa55d97786cb9cfadfe2b59e9.camel@vmware.com>
	<YGaha/euiRjZ820X@paquier.xyz>
	<YGhf4TN6ukOZTUcd@paquier.xyz>
	<ae1143fb-455c-c80f-ed66-78d45bd93303@enterprisedb.com>

On Wed, Sep 22, 2021 at 08:59:38AM +0200, Peter Eisentraut wrote:
> I noticed this patch eliminated one $Test::Builder::Level assignment. Was
> there a reason for this?
> 
> I think we should add it back, and also add a few missing ones in similar
> places.  See attached patch.
>
> [...]
>
>  {
> +	local $Test::Builder::Level = $Test::Builder::Level + 1;
> +

So you are referring to this one removed in c50624c.  In what does
this addition change things compared to what has been added in
connect_ok() and connect_fails()?  I am pretty sure that I have
removed this one because this logic got refactored in
PostgresNode.pm.
--
Michael

Attachments:

  [application/pgp-signature] signature.asc (832B, ../YUrdnoEc4NPfBZfI@paquier.xyz/2-signature.asc)
  download

view thread (80+ messages)  latest in thread

Message-ID: <YUrdnoEc4NPfBZfI@paquier.xyz>
Permalink:  ../YUrdnoEc4NPfBZfI@paquier.xyz/
Also on:    postgresql.org/message-id/YUrdnoEc4NPfBZfI@paquier.xyz

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: michael@paquier.xyz, peter.eisentraut@enterprisedb.com, pchampion@vmware.com
  Subject: Re: Proposal: Save user's original authenticated identity for logging
  In-Reply-To: <YUrdnoEc4NPfBZfI@paquier.xyz>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox