Skip to content

Conversation

@xabg2
Copy link
Collaborator

@xabg2 xabg2 commented Jan 16, 2026

This PR centralizes the logging approach across the webhooks module by replacing the pattern of passing the Fastify logger as a dependency through multiple function calls and class constructors. Instead, we now use the centralized Logger object directly where logging is needed. This eliminates unnecessary parameter drilling and makes the code cleaner and easier to maintain.

Additionally, this refactor removes other unused dependencies that were being passed around without purpose, such as the config object in several webhook handlers and the direct Stripe instance injection in the app setup (now using the existing stripePaymentsAdapter singleton instead). The webhook entry functions have also been renamed to be more descriptive (providerWebhooksEvents, cryptoProviderWebhooks) and their signatures updated to use a configuration object instead of positional parameters for better readability.

@xabg2 xabg2 self-assigned this Jan 16, 2026
@xabg2 xabg2 added the enhancement New feature or request label Jan 16, 2026
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
66.7% Coverage on New Code (required ≥ 85%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants