From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate001.proxmox.com (gate001.proxmox.com [45.144.208.40]) by lore.proxmox.com (Postfix) with ESMTPS id 1FB6A1FF09B for ; Mon, 17 Aug 2026 14:58:07 +0200 (CEST) Received: from gate001.proxmox.com (localhost.localdomain [127.0.0.1]) by gate001.proxmox.com (Proxmox) with ESMTP id AC14721768; Mon, 17 Aug 2026 14:57:55 +0200 (CEST) From: Lukas Wagner To: pdm-devel@lists.proxmox.com Subject: [PATCH datacenter-manager 19/20] pdm-config: subscriptions: drop unused accessor functions Date: Mon, 17 Aug 2026 14:57:26 +0200 Message-ID: <20260817125727.454039-20-l.wagner@proxmox.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260817125727.454039-1-l.wagner@proxmox.com> References: <20260817125727.454039-1-l.wagner@proxmox.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1786971437447 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.897 Adjusted score from AWL reputation of From: address DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment (newer systems) RCVD_IN_DNSWL_MED -2.3 Sender listed at https://www.dnswl.org/, medium trust SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Message-ID-Hash: IIMGVSLVNLFRO63X43NRUTRIS6JSSKAN X-Message-ID-Hash: IIMGVSLVNLFRO63X43NRUTRIS6JSSKAN X-MailFrom: l.wagner@proxmox.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header X-Mailman-Version: 3.3.10 Precedence: list List-Id: Proxmox Datacenter Manager development discussion List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: Now that the subscription API handlers reach the config through PdmApplication instead, the free functions built on top of the global INSTANCE static are unused. Drop them along with the init() call that used to populate it. Signed-off-by: Lukas Wagner --- lib/pdm-config/src/subscriptions.rs | 39 ----------------------------- server/src/context/mod.rs | 1 - 2 files changed, 40 deletions(-) diff --git a/lib/pdm-config/src/subscriptions.rs b/lib/pdm-config/src/subscriptions.rs index 5be88a13..c4b99d25 100644 --- a/lib/pdm-config/src/subscriptions.rs +++ b/lib/pdm-config/src/subscriptions.rs @@ -7,8 +7,6 @@ //! entries, which is intended as future proofing for a more automated (shop) import without having //! to adapt the data layer. -use std::sync::OnceLock; - use anyhow::Error; use proxmox_config_digest::ConfigDigest; @@ -25,37 +23,6 @@ pub const SUBSCRIPTIONS_CFG_FILENAME: &str = configdir!("/subscriptions/keys.cfg const SUBSCRIPTIONS_SHADOW_FILENAME: &str = configdir!("/subscriptions/keys.shadow"); pub const SUBSCRIPTIONS_CFG_LOCKFILE: &str = configdir!("/subscriptions/.keys.lock"); -static INSTANCE: OnceLock> = OnceLock::new(); - -fn instance() -> &'static (dyn SubscriptionKeyConfig + Send + Sync) { - INSTANCE - .get() - .expect("subscription key config not initialized") - .as_ref() -} - -pub fn lock_config() -> Result { - instance().lock() -} - -pub fn config() -> Result<(SectionConfigData, ConfigDigest), Error> { - instance().read() -} - -pub fn shadow_config() -> Result, Error> { - instance().read_shadow() -} - -pub fn save_config( - config: &SectionConfigData, -) -> Result { - instance().write(config) -} - -pub fn save_shadow(shadow: &SectionConfigData) -> Result<(), Error> { - instance().write_shadow(shadow) -} - pub trait SubscriptionKeyConfig { fn read(&self) -> Result<(SectionConfigData, ConfigDigest), Error>; fn read_shadow(&self) -> Result, Error>; @@ -110,9 +77,3 @@ impl SubscriptionKeyConfig for DefaultSubscriptionKeyConfig { replace_secret_config(SUBSCRIPTIONS_SHADOW_FILENAME, raw.as_bytes()) } } - -pub fn init(instance: Box) { - if INSTANCE.set(instance).is_err() { - panic!("subscription key config instance already set"); - } -} diff --git a/server/src/context/mod.rs b/server/src/context/mod.rs index 5f95d358..b64275c8 100644 --- a/server/src/context/mod.rs +++ b/server/src/context/mod.rs @@ -36,7 +36,6 @@ pub fn init() -> Result { // anyway, and the implementation is stateless, so having this second // instance is not an issue *currently*. pdm_config::remotes::init(factory.make_remote_config()?); - pdm_config::subscriptions::init(factory.make_subscription_key_config()?); Ok(app) } -- 2.47.3