Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1sI1cN-00CGAj-3R for pgsql-general@arkaria.postgresql.org; Fri, 14 Jun 2024 07:45:03 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.94.2) (envelope-from ) id 1sI1cK-00EsLF-VG for pgsql-general@arkaria.postgresql.org; Fri, 14 Jun 2024 07:45:01 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1sI1cK-00EsKc-KZ for pgsql-general@lists.postgresql.org; Fri, 14 Jun 2024 07:45:01 +0000 Received: from mail-ed1-x52d.google.com ([2a00:1450:4864:20::52d]) by magus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (Exim 4.94.2) (envelope-from ) id 1sI1cJ-001b6Q-7B for pgsql-general@lists.postgresql.org; Fri, 14 Jun 2024 07:45:00 +0000 Received: by mail-ed1-x52d.google.com with SMTP id 4fb4d7f45d1cf-57c68682d1aso1846224a12.3 for ; Fri, 14 Jun 2024 00:44:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cybertec-at.20230601.gappssmtp.com; s=20230601; t=1718351098; x=1718955898; darn=lists.postgresql.org; h=mime-version:user-agent:content-transfer-encoding:autocrypt :references:in-reply-to:date:cc:to:from:subject:message-id:from:to :cc:subject:date:message-id:reply-to; bh=5ETBD5CpRHvTApxmMEWSWSyf+w4/MpvFDk+ki6KLMP8=; b=KDeQ3aVyj3xWGNC7iRrP1kN2Fixn+c/hBSVHPqvcmZRJv/CmuEM+Wbqqebg2QwiI/f TWIG2l5lNGlNUZ+IQl9PTy8ukDhbbLC5qbUszD6t8Q84qG9gq0IfxCpABMsAOMH3VAmC +FgjE0NZPyQAo1oeDzkZg5djeTQNBSGaFHxYmkIda/DBTl5IoyD6ngQ/0KJEUB9m4bzZ XcxyiZ/99OIBwQGL1p+PqrAeuT4cIPD27N0MtGisuFfjWtl9+zJCl9K2E74QykPA1cir CzZw3EOD7VqWQ4WufLDjIjWWGNJbJhlUjcgwRm9COaEPpc104yPkXHs3+hjlVnIH9J9/ Bxsg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1718351098; x=1718955898; h=mime-version:user-agent:content-transfer-encoding:autocrypt :references:in-reply-to:date:cc:to:from:subject:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=5ETBD5CpRHvTApxmMEWSWSyf+w4/MpvFDk+ki6KLMP8=; b=lU4mv3v95bgYvDIyl81DmvdcEkPE9DEukMtGRMpnu6NglZezTCs8B1ZkIju79bHGW/ 78ictDH5Qq+jJXuLTizdGLNn/UzJ9eXkQLH/1hY3osae2D00QZxS1vjiYRT6lImDa1aF 3eTUo/fAj+5CPP71HbaQiQtbAcHLd+8tKulKazVJnqA5tFBuVGVCqajXwpq+NZArjnIG WU5sEgv2vJWU6f5m7n8HY9fXtdobcyRdNw55UG3l5zHLpuIsT+tw21SNt4JwiYKLCOV3 C1FH+oWbURBUtabz23rTrHyBUNHng7H4zibUbDtDjY9sxPki9ge83jZPty0Y7drKMXm4 X0Hg== X-Forwarded-Encrypted: i=1; AJvYcCVa4nPbbkiYKZMYgpBsGff5S7L6xakvm/36ztgxFuN9b4nuLMufJB5cSCGGhMV5eUgxo6wNU6hkAWZwftq16GBT+EZjnCEkyjuzhKTuQNEn2ApR X-Gm-Message-State: AOJu0YzM39XUaux804D1y+I3P/TaubS9Mw/UdQzQFFu491HVEQdW+Lfp vJ6FqXK24NBCwIy2OD/I0Wgo9HmJGlO0uT9t9qDVW2U4TK5j8bIDakLNBHMn244= X-Google-Smtp-Source: AGHT+IHjhSkqoD4+2KNwZTgRNXuy51eHdbOGdOaU6XCZmUR0gzJ5QISgOoGord8ehI7CYSrTy9EHlg== X-Received: by 2002:a50:d6de:0:b0:57c:6248:62b7 with SMTP id 4fb4d7f45d1cf-57cbd663a9amr1240855a12.13.1718351098353; Fri, 14 Jun 2024 00:44:58 -0700 (PDT) Received: from localhost.localdomain ([2001:871:260:b84e:45b4:e7f8:160c:4b00]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-57cb72e989esm1902477a12.49.2024.06.14.00.44.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 14 Jun 2024 00:44:58 -0700 (PDT) Message-ID: <4c63472a57a2b5843cbae3e4020216b433004861.camel@cybertec.at> Subject: Re: Reserving GUC prefixes from a non-preloaded DB extension is not always enforced From: Laurenz Albe To: Narek Galstyan , pgsql-general@lists.postgresql.org Cc: narek@lantern.dev Date: Fri, 14 Jun 2024 09:44:57 +0200 In-Reply-To: References: Autocrypt: addr=laurenz.albe@cybertec.at; prefer-encrypt=mutual; keydata=mQINBGGDwAQBEADgbWy5cKXQld3N2mF+DFyiNFbi2oBl2T+XgxpPF8wTRw2D/u4bBKXP0SYSE/lA86jIVNWWU0gf1KODIkVvgJm2w4vH2VBV1b7ddVViGl1Iu+9zaRnv9wulhnH42KefepXnoean6UT1EzLM0opF/Ik0j+40TxdRtobkBprkQUyHDXWlHc2ffPs3SipyFEP9AVLf7ejRC46CXWDnsqjOBSMEW8Z4HiK/8RrPZBsKLts8dJxKF4pygOdJb0CWk8k/X1jbcfdxo+zOLjOMvJcSJ2pFdJmQHU+JufB3rePziqQ2S9Ur6sccr9XnTC1GVBWN4Lf5VHq+vf+bFJjVwg+2hrySZnAVfcOrxoqFLErr7ug1zN2nM1kcpgA4VWn4gxlJtYNYYq+9WxX5dtvnNANlG3ZCrRKQzl8lxtzoF6Zo7LUhEqPaHDwn7Rvs+IdbOn41lF5UDTJGqmC4gS/bZydW2Fy3YWm4aSaN9fgFf8D+PVkrlKAZB7gBLz1TyHjbcRf85cYF+GKKrDld5SzMB/V60VX3oP/Eo8ikFpyWaqiz1f9X7MBot3/PjJkY+wDzp3nmb19QEcOBuQiSQ4xds2r0HewbuHTAR68u8jNNMGmpm2j4x+g09Jd/WQDjqlTBZ/jEltH41fYCCPWMfljXTOOXu2eLNGdfi7ETZogtwjM9oTtSPQARAQABtCdMYXVyZW56IEFsYmUgPGxhdXJlbnouYWxiZUBjeWJlcnRlYy5hdD6JAk4EEwEIADgWIQR0CqhbZGGABqoaSbdi8bhXA2EdmAUCYYPABAIbAwULCQgHAgYVCgkICwIEFgIDAQIeAQIXgAAKCRBi8bhXA2EdmM/6EADK232JCwmBzhlj8h7U9CjG6kx0JHP3uJGv+XfsHtHAlmY/RCwF1BHMEsRlk bT5UrLvJ2jb99bA9QARzhFaxzyn0F/BUKzuIjRGNs/n6d5dNUFA0kOt8sX+TacmC GEyjEBCrVCm4ranBiUyePn9NhHNWnaex7pJyqvMLLdwW9BEMJx0Fqo+DN8ukbXmYRsmhEtd3ue+x/luYmOmJnaGtzInaY5aOJYbW9XqoRIZkZvOCgbi1FfvNmoqWa+3oVxTOgw9RafjJDyW0lTHzKGjbGI5ofMU98l+/hKJFYJqWUF6VpFJY5YIcN/1lf4ZICMwDl+MPIVo/tpq8L10seJL28nLlvw3K+cI+TVW8IW/qL/LyVoDofI3USeOORuYmhpWRhik8JXX6xf3v6GrRilJIPWNFIJbxm1ZblQiQnOw3IOW7T+8nAmPin1HKqM3VrOrJQ2VtShsefNBibNAsr1oFaqcDBkn3yGG8i6CTW+FyO4PZ+/EwNxMVgktxbYdy5AT1/lpXr5tB+phhLIyVfiBvrWs5EThxYMQ/L8Y85c3GMsAy1l/x4h3jqySIYy3SCU9+jc5UVuNnXljbvkEzJ+NLWJ6C1rACFWrMszgPdh5tCrlRY9PpmYll4JbCgb8BtxEIUmR+xr50/ZElEK5iml7Q00KUekCcDt+36PsyGFTXBzNOrkCDQRhg8AEARAAzOZ2tLHlI4rrhG411h6cdCFjBZxuljaFCxFyHn3m6wbGLqwBUWC5k8UrRqjHMz88KcTSaNO7XGAmCqPdWd2SeflPZRnNTbjsVpw7mLdffsBm4JX7kki2Pvk5h0NtYeidXT1PSpc2ri4DutYXuT9uD8RAm1wUDCE5HQNUihT/WH6opt+hskHW21uHao0+y822tG0QQcGMqdQR5Vxdxj89wiEPdqW+HpU/oOZIhrf2E7prduAppxixjHy/o1rcnoznnJvc8D3+YgI9O0LrBMij89dM55pRGbLovTR1oGR3U74sX774+0xmSzeIKwZfiMUz7Atlvfk5SHOsRUFPN2Ux9kaXiiBibQpHFxt7b lDrT4wxdLJ/XCdbPPAyl+lZtOLsaHEEZvYNyTXwZc35dVf3R4/oz20HoG6s7ct8e1 AQygj43XAERzty9SkWgxs8+grp1PrGx6FHVSYRqBM8dS/ZR6yRVwOwJXPyaSSqfIF21DkE4j1y4n+ItSewPGoRp8K/yWCikt6qlkVkO2ASNIiX04fAbtzwVOaNn8ZMRNqyvLc1fED4sr49onE4cAIcBLjcC3KL+w9DUGRQCdziROj5H2Yl/sXGPdMciUHo/Uz2rggc+2th3bQiMhrHWSsBpUkDQp0yWewemstPpPgBL3h2fHKaX8B9oH5Qu/H1IgrOuX8AEQEAAYkCNgQYAQgAIBYhBHQKqFtkYYAGqhpJt2LxuFcDYR2YBQJhg8AEAhsMAAoJEGLxuFcDYR2YuPwQAMkpGtR80pQ1gVsONhdkqj0H2eU66efP/gO3CoyaoIcvrpKYj7C2HipVSmkt1gpByL0X4AMQ/vKuknUz3wd28Ba+G1dCfbVs/Xiusq+SmpUj5rTwmYqdSjWMuCo1R6oS5hdJMdUUJYGMT0QkVlm1KnW8jkmCTl9GzjDxOAsN9O6/6lPzaGFtk9XF+34Bry/N4HKiJkqpC4+UTd0AprPfzJ2jdT64e1F0+W88X8y1bTTgNrHwK4mDiLnlE4SKRuEm54lNhJz//ar86Or5BErzNpM6TL7lk44QS06hwsMrEdKIy8J/SYJPjfzR8tIUnKscclVpOgjKaBqC+0iFiVaRqAgfOlIEiezX6kMh5Q2FIUfqs46qWhhXjRrdKOEoStYAaikdLu5ZXr7vfb0ZaDh+ZwTQtbSMFolyOkecwI81MCdbMfT/1TqIGTOdAj5as9fAakk0jb2pXgUYQ8X1DVTR8ahSDVEaw9VTmWiSvTxvguVJ1Mb7gG4Gmh6aviDTJhfXtH4rPUNXhDLqrTH8JkJjyKROOMakIF68Hjse5vUfUxreBEOtb5r1Coa2Fe7ncJayaSE7ryrDbFqpZ 36UMAx4ulWMyqJajLNGY0DdG8qIsR5nxRhrnK/mrCidZ8F9/D3bWAl4rjtHlsztN59 +AnW5l0HsQcY9ntFL/zEBOaonjdJf Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.50.4 (3.50.4-1.fc39) MIME-Version: 1.0 List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk On Thu, 2024-06-13 at 12:26 -0700, Narek Galstyan wrote: > I am an extension developer. I use `MarkGUCPrefixReserved` to reserve GUC= prefixes, > which my extension uses to help avoid accidentally misspelled config-file= entries. >=20 > However, since the reservation happens in `_PG_init()` and `_PG_init()` i= s not > called until the first use of an API exposed by my extension, misspelled = config-file > entries that get executed before the extension is loaded will not throw a= n error. >=20 > I'd expect GUC variables reserved by an extension to live more permanentl= y in > Postgres catalogs (e.g., in pg_settings).=C2=A0 > So, even when the extension binary is not loaded, Postgres would know whi= ch prefixes > are reserved and which GUC settings must be allowed (similar to how Postg= res knows > in pg_extension which extensions are enabled, even when the corresponding= extension > binary has not been loaded). >=20 > > 1. Would you consider the proposed behavior an improvement? Not really. If I wanted to avoid that problem, I'd put the extension in "shared_preload= _libraries", so that _PG_init() is executed when the server starts. Yours, Laurenz Albe