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 0BB471FF09F for ; Thu, 03 Sep 2026 08:50:54 +0200 (CEST) Received: from gate001.proxmox.com (localhost.localdomain [127.0.0.1]) by gate001.proxmox.com (Proxmox) with ESMTP id 8112E2163C; Thu, 03 Sep 2026 08:50:42 +0200 (CEST) From: Lukas Wagner To: pdm-devel@lists.proxmox.com Subject: [PATCH datacenter-manager v4 20/21] pdm-config: subscriptions: drop unused accessor functions Date: Thu, 3 Sep 2026 08:50:15 +0200 Message-ID: <20260903065016.31726-21-l.wagner@proxmox.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260903065016.31726-1-l.wagner@proxmox.com> References: <20260903065016.31726-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: 1788418219128 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.524 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: RF6NWSE2IGVJI5CDBKBZZLO2L7HTQPEX X-Message-ID-Hash: RF6NWSE2IGVJI5CDBKBZZLO2L7HTQPEX 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 082ab8de..95238030 100644 --- a/server/src/context/mod.rs +++ b/server/src/context/mod.rs @@ -34,7 +34,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