Open
Conversation
…moving the old handler. Update main application to utilize the new service for wallet webhooks. Clean up configuration by removing unused WebhookURL variable.
📊 Performance MetricsPerformance Metrics (memory storage)
|
…e management. Update various webhook-related functions to handle errors gracefully and improve code clarity by using underscore for ignored return values.
📊 Performance MetricsPerformance Metrics (postgres storage)
|
📊 Performance MetricsPerformance Metrics (cluster-valkey storage)
|
… of response body with error handling.
…ook tests - Added `WALLET_LIST_URL` and `WALLET_LIST_REFRESH_INTERVAL` to `docker-compose.dnsmasq.yml` and `docker-compose.nginx.yml` for wallet integration. - Refactored webhook tests to utilize a shared mock server and improved handling of webhook readiness and record management.
- Changed `WALLET_LIST_URL` from `http://bridge-gointegration:9091/wallets` to `http://gointegration:9091/wallets` across multiple Docker Compose files for consistency. - Updated `WEBHOOK_MOCK_HOST` from `bridge-gointegration` to `gointegration` in relevant services to reflect the new configuration.
📊 Performance MetricsPerformance Metrics (nginx storage)
|
📊 Performance MetricsPerformance Metrics (dnsmasq storage)
|
- Replaced the previous WALLET_LIST_URL and WALLET_LIST_REFRESH_INTERVAL with a new WEBHOOK_CONFIG environment variable for per-wallet webhook settings. - Updated the main application and webhook service to utilize the new configuration format, allowing for individual wallet URLs and optional authentication tokens. - Adjusted Docker Compose files and documentation to reflect these changes, ensuring consistency across services.
…on token - Modified the WEBHOOK_CONFIG environment variable across multiple Docker Compose files to include an authentication token for the test wallet. - Updated related tests to validate the presence of the authorization header in webhook requests.
… and improve readability.
…onfiguration - Updated the webhook service to allow configuration from both inline JSON and an optional file/URL source. - Introduced a refresh mechanism for source-backed configurations, ensuring updates are applied periodically. - Enhanced documentation to reflect the new configuration options and their usage. - Added tests to validate loading and merging of webhook configurations from different sources.
…or notifications - Modified the webhook service to ensure that notifications are only sent when both `wallet` and `topic` query parameters are present. - Updated related documentation to clarify the new requirement for webhook payloads. - Refactored tests to validate the behavior when the `topic` parameter is omitted, ensuring no notifications are sent in such cases.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
No description provided.