fix: resolve inconsistent console usage in logging implementation

- 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 <noreply@anthropic.com>
This commit is contained in:
2025-09-20 21:21:35 +02:00
parent da96a0a838
commit b27b5806b1
2 changed files with 8 additions and 7 deletions

View File

@@ -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) => { onInit: async (payload: Payload) => {
@@ -204,6 +201,12 @@ export const stripeProvider = (stripeConfig: StripeProviderConfig) => {
apiVersion: stripeConfig.apiVersion || DEFAULT_API_VERSION, apiVersion: stripeConfig.apiVersion || DEFAULT_API_VERSION,
}) })
singleton.set(payload, stripe) 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) => { initPayment: async (payload, payment) => {
// Validate required fields // Validate required fields

View File

@@ -928,14 +928,12 @@ function generateTestPaymentUI(
setTimeout(() => pollStatus(), 2000); setTimeout(() => pollStatus(), 2000);
} }
} catch (error) { } catch (error) {
const logger = createContextLogger(payload, 'Test Provider') console.error('[Test Provider] Failed to poll status:', error);
logger.error('Failed to poll status:', error);
} }
} }
${testModeIndicators.consoleWarnings !== false ? ` ${testModeIndicators.consoleWarnings !== false ? `
const logger = createContextLogger(payload, 'Test Provider') console.warn('[Test Provider] 🧪 TEST MODE: This is a simulated payment interface for development purposes');
logger.warn('🧪 TEST MODE: This is a simulated payment interface for development purposes');
` : ''} ` : ''}
</script> </script>
</body> </body>