-
Notifications
You must be signed in to change notification settings - Fork 32
Send Ton Analytics events #162
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
📊 Performance MetricsPerformance Metrics (postgres storage)
Performance Metrics (memory storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (nginx storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR integrates Ton Analytics event tracking throughout the bridge service. The changes refactor the analytics client to use instance methods instead of standalone functions, add comprehensive event tracking for bridge operations (connection lifecycle, message handling, verification), and introduce helper utilities for normalizing client IDs and handling optional fields.
- Refactored analytics client from standalone functions to instance methods with configuration fields
- Added 11 new event types covering the complete bridge operation lifecycle
- Integrated analytics event sending across v1 and v3 handlers for all major operations
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| tonmetrics/analytics.go | Refactored analytics client to use instance methods, added 11 event creation methods, and implemented helper functions for optional fields and event ID generation |
| internal/v3/handler/handler.go | Added analytics client field and event tracking for connection lifecycle, message operations, and validation errors; introduced normalizeClientIDs helper |
| internal/v1/handler/handler.go | Similar analytics integration as v3 with event tracking for all major operations and normalizeClientIDs helper |
| internal/v1/storage/pg.go | Added expired message event tracking in the worker |
| internal/v1/storage/mem.go | Added expired message event tracking and analytics client initialization |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (nginx storage) ❌ Bridge service not accessible after 30 seconds |
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (nginx storage) ❌ Bridge service not accessible after 30 seconds |
acf5df2 to
a2339b6
Compare
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (nginx storage) ❌ Bridge service not accessible after 30 seconds |
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds |
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds |
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (nginx storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds |
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (nginx storage) ❌ Bridge service not accessible after 30 seconds |
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (nginx storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds |
1f15bbc to
80e47f9
Compare
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (nginx storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds |
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (nginx storage) ❌ Bridge service not accessible after 30 seconds |
fa2c7f7 to
e04349e
Compare
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
Performance Metrics (cluster-valkey storage)
Performance Metrics (nginx storage)
Performance Metrics (dnsmasq storage)
|
📊 Performance MetricsPerformance Metrics (postgres storage)
Performance Metrics (memory storage)
Performance Metrics (cluster-valkey storage)
Performance Metrics (nginx storage)
Performance Metrics (dnsmasq storage)
|
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
Performance Metrics (cluster-valkey storage)
Performance Metrics (dnsmasq storage)
Performance Metrics (nginx storage)
Performance Metrics (nginx storage)
Performance Metrics (dnsmasq storage)
|
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
Performance Metrics (cluster-valkey storage)
Performance Metrics (nginx storage)
Performance Metrics (dnsmasq storage)
|
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
Performance Metrics (cluster-valkey storage)
Performance Metrics (dnsmasq storage)
Performance Metrics (nginx storage)
|
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
|
869e650 to
d89e39f
Compare
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
Performance Metrics (cluster-valkey storage)
Performance Metrics (dnsmasq storage)
Performance Metrics (nginx storage)
|
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
Performance Metrics (cluster-valkey storage)
Performance Metrics (dnsmasq storage)
Performance Metrics (nginx storage)
|
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
Performance Metrics (cluster-valkey storage)
Performance Metrics (nginx storage)
Performance Metrics (dnsmasq storage)
|
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
Performance Metrics (cluster-valkey storage)
Performance Metrics (nginx storage)
Performance Metrics (dnsmasq storage)
|
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
Performance Metrics (cluster-valkey storage)
Performance Metrics (nginx storage)
Performance Metrics (dnsmasq storage)
|
📊 Performance MetricsPerformance Metrics (memory storage)
|
📊 Performance MetricsPerformance Metrics (memory storage)
|
📊 Performance MetricsPerformance Metrics (postgres storage)
|
📊 Performance MetricsPerformance Metrics (cluster-valkey storage)
|
📊 Performance MetricsPerformance Metrics (nginx storage)
|
📊 Performance MetricsPerformance Metrics (dnsmasq storage)
|
📊 Performance MetricsPerformance Metrics (dnsmasq storage)
|
📊 Performance MetricsPerformance Metrics (memory storage)
|
📊 Performance MetricsPerformance Metrics (postgres storage)
|
📊 Performance MetricsPerformance Metrics (cluster-valkey storage)
|
📊 Performance MetricsPerformance Metrics (dnsmasq storage)
|
📊 Performance MetricsPerformance Metrics (nginx storage)
|
📊 Performance MetricsPerformance Metrics (memory storage)
|
📊 Performance MetricsPerformance Metrics (postgres storage)
|
📊 Performance MetricsPerformance Metrics (nginx storage)
|
📊 Performance MetricsPerformance Metrics (cluster-valkey storage)
|
📊 Performance MetricsPerformance Metrics (dnsmasq storage)
|
…idge into ton-analytics-send-events
📊 Performance MetricsPerformance Metrics (memory storage)
|
📊 Performance MetricsPerformance Metrics (postgres storage)
|
📊 Performance MetricsPerformance Metrics (nginx storage)
|
📊 Performance MetricsPerformance Metrics (cluster-valkey storage)
|
📊 Performance MetricsPerformance Metrics (dnsmasq storage)
|
No description provided.