From b27b5806b1bbaf29c673cc3df03ccc68ed1ce790 Mon Sep 17 00:00:00 2001 From: Bas van den Aakster Date: Sat, 20 Sep 2025 21:21:35 +0200 Subject: [PATCH] fix: resolve inconsistent console usage in logging implementation MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Move Stripe provider webhook warning to onInit where payload is available - Fix client-side logging in test provider UI generation - Replace server-side logger calls with browser-compatible console in generated HTML - Maintain proper logging context separation between server and client code 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- src/providers/stripe.ts | 9 ++++++--- src/providers/test.ts | 6 ++---- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/src/providers/stripe.ts b/src/providers/stripe.ts index 186bd9e..b6854cf 100644 --- a/src/providers/stripe.ts +++ b/src/providers/stripe.ts @@ -193,9 +193,6 @@ export const stripeProvider = (stripeConfig: StripeProviderConfig) => { } } ] - } else { - // Log that webhook endpoint is not registered - console.warn('[Stripe Provider] Webhook endpoint not registered - webhookSecret not configured') } }, onInit: async (payload: Payload) => { @@ -204,6 +201,12 @@ export const stripeProvider = (stripeConfig: StripeProviderConfig) => { apiVersion: stripeConfig.apiVersion || DEFAULT_API_VERSION, }) singleton.set(payload, stripe) + + // Log webhook registration status + if (!stripeConfig.webhookSecret) { + const logger = createContextLogger(payload, 'Stripe Provider') + logger.warn('Webhook endpoint not registered - webhookSecret not configured') + } }, initPayment: async (payload, payment) => { // Validate required fields diff --git a/src/providers/test.ts b/src/providers/test.ts index 6533b73..7e857bb 100644 --- a/src/providers/test.ts +++ b/src/providers/test.ts @@ -928,14 +928,12 @@ function generateTestPaymentUI( setTimeout(() => pollStatus(), 2000); } } catch (error) { - const logger = createContextLogger(payload, 'Test Provider') - logger.error('Failed to poll status:', error); + console.error('[Test Provider] Failed to poll status:', error); } } ${testModeIndicators.consoleWarnings !== false ? ` - const logger = createContextLogger(payload, 'Test Provider') - logger.warn('🧪 TEST MODE: This is a simulated payment interface for development purposes'); + console.warn('[Test Provider] 🧪 TEST MODE: This is a simulated payment interface for development purposes'); ` : ''}