feat: implement live logs with client-side polling #665
Conversation
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
- Add live mode toggle for real-time log streaming - Replace WebSocket approach with HTTP polling every 2 seconds - Implement timeframe-based queries to fetch only new entries - Support both application-specific and global logs views - Reduce ClickHouse load compared to frequent WebSocket queries - Maintain same user experience with more efficient backend approach Technical details: - AppLogs.vue: Use $api.getLogs with time range parameters - Logs.vue: Use $api.getOverview with time range in query JSON - Remove WebSocket endpoints and dependencies from backend - Client manages timestamp tracking to avoid duplicates - 2-second polling interval balances real-time feel with performance
- Run prettier on Vue files (AppLogs.vue, Logs.vue) - Run gofmt on Go files (api/api.go) - Fix code style issues identified by CI/CD pipeline No functional changes, only formatting adjustments to pass automated code quality checks.
|
Thank you for the contribution. |
- Remove unnecessary comment in api/api.go - Remove unnecessary blank lines in main.go
|
Done @apetruhin |
|
Thanks for working on this. We've decided to implement this feature ourselves, as it requires some server-side changes. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Technical details: