Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1nuGEH-0001xp-Po for pgsql-hackers@arkaria.postgresql.org; Thu, 26 May 2022 16:20:53 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1nuGEF-0000VZ-Pf for pgsql-hackers@arkaria.postgresql.org; Thu, 26 May 2022 16:20:51 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1nuGEF-0000RF-E0 for pgsql-hackers@lists.postgresql.org; Thu, 26 May 2022 16:20:51 +0000 Received: from mail-il1-x133.google.com ([2607:f8b0:4864:20::133]) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1nuGEA-0000pN-Uo for pgsql-hackers@postgresql.org; Thu, 26 May 2022 16:20:48 +0000 Received: by mail-il1-x133.google.com with SMTP id j15so1380519ilo.5 for ; Thu, 26 May 2022 09:20:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=BJv0DuuOf3nfkKEOc/2sbXOoGQlBegKNfPbtxIYc8JM=; b=MqOtgbome9/4g2q+a+EIo+NA+6VcAOLU7MUZ23puopHFlI0Mwi+yUCVAIHUFa8JVkj CJfQDreYeF3V9ZPgug1p4O7Pe4QIavTSNv2oBloG08g2o+VIPMLvgEDhpSREwJPY5FUz Z+k7yIZOI+CfbGcfHFtvuJWzXRBOd05Xkm2Ncptv1IYWM8hgUgSeF9jQhFN7DgMn6sdq ubCmTezmhev3iQMGVeFnnGb9+Wyco4rTAhgt11ev3ToPxv1ALj7SeExtl7DzRkdVk3It zp69SWX3v61O6xy/E7H8hKJ1RZoDvA/pe4hAWIMNuwa3kTHcxrxXSB7Tvus0QeLa/zFz 4PUQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=BJv0DuuOf3nfkKEOc/2sbXOoGQlBegKNfPbtxIYc8JM=; b=UyMTansLWsU58n3ax+WNG8vY6ZvzAGycgxARUps1fbO4163gHof24BM9zrMvuQO2t1 nzvtqWV6uMOOCHS1xFPVIODLYp/ejBrkQeuoKZnl/m1JuBSXWzeULAuV+We9yWbda1Ki LLlsxoTkHQl5AYdIsu0onQJsU33RR2128d2YoJlkwVLPlz9ZbGmye4blr+q/w/NzvMph tST2Cn8MKbypCHBChl/Qt/O6tNv1VKHA3vsDibJPsHnm2mxuiVc2xPBSssr5JOV4l8sQ Co0o5HwKaJA6MWd2cs/DZPKXSlWZVLZZK8R6LOdclChHXJL2/dyvb1ZRkq1ICW+is+RT WiyA== X-Gm-Message-State: AOAM53108TACu+grPIXEpHLHW8tSD1eEPD0fUuo9X9768THqvXyFomsp aP08ugj00Lo6/RVwsgqoK+s= X-Google-Smtp-Source: ABdhPJz7bMpXDKB2tKiovj+Y4hr/CDptHXQMwdMmuLrYwsojg6lRuBF8W3Ny1oTQ6SH3yO3b/7Owrg== X-Received: by 2002:a05:6e02:1c29:b0:2d1:7a01:a7db with SMTP id m9-20020a056e021c2900b002d17a01a7dbmr15531943ilh.176.1653582045344; Thu, 26 May 2022 09:20:45 -0700 (PDT) Received: from nathanxps13 ([2600:1700:48f8:340f:cb26:81a2:2ab4:6159]) by smtp.gmail.com with ESMTPSA id n24-20020a6b7218000000b0065a47e16f53sm531436ioc.37.2022.05.26.09.20.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 26 May 2022 09:20:44 -0700 (PDT) Date: Thu, 26 May 2022 09:20:37 -0700 From: Nathan Bossart To: Shinya Kato Cc: Michael Paquier , Kyotaro Horiguchi , przemyslaw@sztoch.pl, david.g.johnston@gmail.com, robertmhaas@gmail.com, daniel@yesql.se, pgsql-hackers@postgresql.org Subject: Re: Add --{no-,}bypassrls flags to createuser Message-ID: <20220526162037.GA1345023@nathanxps13> References: <0738cb3a8a52e850dd17a0065f2d8e5b@oss.nttdata.com> <20220524170910.GA1190535@nathanxps13> <20220525.110752.305692011781436338.horikyota.ntt@gmail.com> <6a25c0fec85c61c0bb93ab4819fdd619@oss.nttdata.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6a25c0fec85c61c0bb93ab4819fdd619@oss.nttdata.com> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk On Thu, May 26, 2022 at 02:16:37PM +0900, Shinya Kato wrote: > On 2022-05-25 11:07, Kyotaro Horiguchi wrote: >> At Tue, 24 May 2022 10:09:10 -0700, Nathan Bossart >> wrote in >> > We're still missing some "fancier" string patterns in the tests, but >> > we >> > might just be nitpicking at this point. >> >> Such "fancier" strings should be properly handled by FmtId() and >> appendStringLiteralConn. If this is a privilege escalating command, >> we should have ones but this is not. > > Sorry, I didn't quite understand the "fancier" pattern. Is a string like > this patch correct? Yes, thanks. I'm marking this as ready-for-committer. -- Nathan Bossart Amazon Web Services: https://aws.amazon.com