[_]: refactor/webhook cleanup #346
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.


This PR centralizes the logging approach across the webhooks module by replacing the pattern of passing the
Fastify loggeras a dependency through multiple function calls and class constructors. Instead, we now use the centralizedLoggerobject 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
configobject in several webhook handlers and the direct Stripe instance injection in the app setup (now using the existingstripePaymentsAdaptersingleton 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.